Re: [PATCH 2/2] gdiplus: Implement GdipBitmapGetHistogram()

[Vincent Povirk]

> - ok(ch1[0x3f] == WIDTH, "Got green (0x3f) %u\n", ch1[0x3f]);
> - ok(ch2[0x20] == WIDTH, "Got blue (0x20) %u\n", ch1[0x20]);
> -
>
> +return;
> -
> - /* ARGB histogram from RGB data. */
> - stat = pGdipBitmapGetHistogram(bm, HistogramFormatARGB, 256, ch0, ch1, ch2, NULL);
> - expect(InvalidParameter, stat);

That return doesn't look like it's supposed to be there.

[PATCH 2/2] gdiplus: Implement GdipBitmapGetHistogram()

[Nikolay Sivov]

On 02.11.2016 0:03, Vincent Povirk wrote:
> > - ok(ch1[0x3f] == WIDTH, "Got green (0x3f) %u\n", ch1[0x3f]);
> > - ok(ch2[0x20] == WIDTH, "Got blue (0x20) %u\n", ch1[0x20]);
> > -
> >
> > +return;
> > -
> > - /* ARGB histogram from RGB data. */
> > - stat = pGdipBitmapGetHistogram(bm, HistogramFormatARGB, 256, ch0, ch1, ch2, NULL);
> > - expect(InvalidParameter, stat);
>
> That return doesn't look like it's supposed to be there.

Oh, right. Thanks.