2009/1/8 Hans Leidekker hans@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.