Re: [5/5] gdiplus: Test flags of loaded images.

Hi,

While running your changed tests on Windows, I think I found new failures.
Being a bot and all I'm not very good at pattern recognition, so I might be
wrong, but could you please double-check?
Full results can be found at
http://testbot.winehq.org/JobDetails.pl?Key=4650

Your paranoid android.

=== WVISTAADM (32 bit image) ===
No test summary line found

=== W7PRO (32 bit image) ===
image.c:103: Test failed: Expected 00012010, got 00013010
image.c:103: Test failed: Expected 00013088, got 00013010

=== W7PROX64 (32 bit image) ===
image.c:103: Test failed: Expected 00012010, got 00013010
image.c:103: Test failed: Expected 00013088, got 00013010

=== W7PROX64 (64 bit image) ===
image.c:103: Test failed: Expected 00012010, got 00013010
image.c:103: Test failed: Expected 00013088, got 00013010

[5/5] gdiplus: Test flags of loaded images.

[Vincent Povirk]

Yep, this patch should not be committed.

So the flags for a loaded image on Windows 7 are always 13010. So
while earlier versions of gdiplus could determine whether the DPI was
set in an image header, win7's gdiplus cannot, and always claims that
it is.

I remember reading that gdiplus on win7 uses WIC for its image
loading/saving, and I know that WIC cannot get this information. So it
makes sense.

It's also very convenient for me.

On Sun, Aug 22, 2010 at 1:42 PM, Marvin  wrote:
> Hi,
>
> While running your changed tests on Windows, I think I found new failures.
> Being a bot and all I'm not very good at pattern recognition, so I might be
> wrong, but could you please double-check?
> Full results can be found at
> http://testbot.winehq.org/JobDetails.pl?Key=4650
>
> Your paranoid android.
>
> === WVISTAADM (32 bit image) ===
> No test summary line found
>
> === W7PRO (32 bit image) ===
> image.c:103: Test failed: Expected 00012010, got 00013010
> image.c:103: Test failed: Expected 00013088, got 00013010
>
> === W7PROX64 (32 bit image) ===
> image.c:103: Test failed: Expected 00012010, got 00013010
> image.c:103: Test failed: Expected 00013088, got 00013010
>
> === W7PROX64 (64 bit image) ===
> image.c:103: Test failed: Expected 00012010, got 00013010
> image.c:103: Test failed: Expected 00013088, got 00013010