Re: [PATCH] winex11: Initialize biCompression in create_icon_pixmaps (Valgrind)

[Alex Henrie]

On Tue, Oct 15, 2019 at 9:26 AM Alexandre Julliard <julliard(a)winehq.org> wrote:
> Alex Henrie <alexhenrie24(a)gmail.com> writes:
> > create_icon_pixmaps calls GetDIBits, which checks biCompression and
> > returns early if the value is incorrect.
>
> No, the code may not be obvious, but it does the right thing.

You're right, create_icon_pixmaps calls GetDIBits with bits set to
NULL, so biCompression is never checked. Thanks for noticing that.
Sadly it means that something even more obscure is causing the
Valgrind warnings :-(

-Alex