Re: [PATCH 1/2] gdiplus/test: Add GdipGetNearestColor test [Try 3]

[Vincent Povirk]

> +        status = GdipGetNearestColor(graphics, &color);
> +        expect(Ok, status);

I think you still need to mark this as todo_wine.

[PATCH 1/2] gdiplus/test: Add GdipGetNearestColor test [Try 3]

[Justin Chevrier]

Do I need that for the 1,4 and 8bpp tests? With my patch to
GdipGetNearestColor these tests pass and behave like Windows 7. I did
forget a todo_wine on the 16bpp grayscale test though...

On Thu, Feb 25, 2010 at 7:53 PM, Vincent Povirk
<madewokherd+8cd9(a)gmail.com> wrote:
> > +        status = GdipGetNearestColor(graphics, &color);
> > +        expect(Ok, status);
>
> I think you still need to mark this as todo_wine.

[PATCH 1/2] gdiplus/test: Add GdipGetNearestColor test [Try 3]

[Vincent Povirk]

> Do I need that for the 1,4 and 8bpp tests? With my patch to
> GdipGetNearestColor these tests pass and behave like Windows 7. I did
> forget a todo_wine on the 16bpp grayscale test though...

The tests should pass independently after each patch in the series is
applied. The todo_wine's are in case the first patch is accepted and
the second is not.

You'll have to remove the todo_wine's from the newly-succeeding tests
in your second patch, of course.