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.