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
 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.