setupapi/tests: Remove an unneeded cast.

Henri Verbeet hverbeet at gmail.com
Thu Jan 8 03:26:32 CST 2009


2009/1/8 Hans Leidekker <hans at codeweavers.com>:
> On Thursday 08 January 2009 00:05:01 Francois Gouget wrote:
>
>>      memset(buffer, 0x55, sizeof(buffer));
>> -    retval = pStringTableLookUpStringEx(table, uilevel, ST_CASE_SENSITIVE_COMPARE, buffer, (LPDWORD)0);
>> +    retval = pStringTableLookUpStringEx(table, uilevel, ST_CASE_SENSITIVE_COMPARE, buffer, NULL);
>>      ok(retval != ~0u, "failed find 'UILEVEL' in string table\n");
>>      ok(memcmp(buffer, &data, 4), "unexpected data\n");
>
> This test is there to show that the prototype of StringTableLookUpStringEx
> is broken; the last parameter is treated as an integer instead of a pointer
> and this notation serves to document that fact. But I can see how this will
> repeatedly trigger janitorial scripts, so I do not object to this patch
> going in.
>
Using 0 instead of NULL should work just as well, if that makes it any clearer.



More information about the wine-devel mailing list