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

[Dmitry Timoshkov]

Nikolay Sivov <nsivov(a)codeweavers.com> wrote:
> +    for (x = 0; x < width; x++)
> +        for (y = 0; y < height; y++)
> +        {
> +            ARGB color;
> +
> +            GdipBitmapGetPixel(bitmap, x, y, &color);
> +            set_histogram_point[format](color, ch0, ch1, ch2, ch3);
> +        }
> +
> +    return Ok;
> +}

One of the very least things that could be done about this very ineffecient
implementation is to make the width traversing an inner loop.

-- 
Dmitry.