[1/5] comctl32/tests: more tests for icon spacing calculation
Daniel Jelinski
djelinski1 at gmail.com
Fri Jan 25 02:47:35 CST 2013
Hello,
Thanks for reviewing. What's the preferred way of checking whether the
code is compiled for 32 or 64bit?
Regards,
Daniel
2013/1/24 Alexandre Julliard <julliard at winehq.org>:
> Daniel Jelinski <djelinski1 at gmail.com> writes:
>
>> +#ifdef _WIN64
>> + /* NOTE: -1 is not treated the same as (DWORD)-1 by 64bit listview */
>> + ret = SendMessage(hwnd, LVM_SETICONSPACING, 0, (DWORD)-1);
>> +todo_wine {
>> + expect(100, LOWORD(ret));
>> + expect(0xFFFF, HIWORD(ret));
>> +}
>> + ret = SendMessage(hwnd, LVM_SETICONSPACING, 0, 0xBAADF00DDEADBEEFLL);
>
> Please avoid ifdefs and long long constants.
>
> --
> Alexandre Julliard
> julliard at winehq.org
More information about the wine-devel
mailing list