[3/5] comctl32/dpa: Implement DPA_SaveStream() with tests
Nikolay Sivov
bunglehead at gmail.com
Mon Sep 14 02:04:07 CDT 2009
Vitaliy Margolen wrote:
> Nikolay Sivov wrote:
>
>> Changelog:
>> - Implement DPA_SaveStream() with tests
>>
>> + streamData.dwSize = curr_pos.QuadPart - initial_pos.QuadPart;
>> + streamData.dwData2 = 1;
>> + streamData.dwItems = streamInfo.iPos;// + 1;
>>
> c++ comment.
>
>
>> +#if 0
>> + /* crashes on XP */
>> + hRes = pDPA_SaveStream(NULL, CB_Save, pStm, NULL);
>> + expect(E_INVALIDARG, hRes);
>> +
>> + hRes = pDPA_SaveStream(dpa, CB_Save, NULL, NULL);
>> + expect(E_INVALIDARG, hRes);
>> +#endif
>>
> Don't use #if 0 / #endif in tests. Use if (0){} instead. This will guarantee
> that code will still compile.
>
> Vitaliy.
>
Thanks.
More information about the wine-devel
mailing list