comctl32/tests: Add two casts to avoid comparisons of differentint types.
Janne Hakonen
joyer83 at live.fi
Sun Feb 13 14:00:58 CST 2011
> Hmm, looking at this my patch changes
>
> ok(*(ULONG_PTR*)g_drawitem.itemData == lparam,
> "got %lx, expected %lx\n", gdrawitem.itemData, lparam);
>
> to
>
> ok(*(ULONG_PTR*)g_drawitem.itemData == (ULONG)lparam,
> "got %lx, expected %lx\n", g_drawitem.itemData, lparam);
>
> That is, the only difference is the (ULONG) cast. Given the output above,
> my patch did not trigger this failure since based on the printing of the
> two values they are different.
Actually, now with your change you are comparing ULONG_PTR to ULONG.
With 64 bit binary the left side of comparison is a 64 bit pointer and right
side is 32 bit value.
Best Regards,
Janne Hakonen
More information about the wine-devel
mailing list