[PATCH 1/6] lz32/tests: Test more last errors set by LZOpenFile[AW].

Michael Stefaniuc mstefani at redhat.com
Tue Oct 10 13:16:53 CDT 2006


James Hawkins wrote:
> On 10/10/06, Saulius Krasuckas <saulius.krasuckas at ieee.org> wrote:
> 
>> @@ -148,6 +149,8 @@ static void test_LZOpenFileA_existing_co
>>         test.szPathName, expected);
>>      LZClose(file);
>>    } else { /* Win9x */
>> +    ok(GetLastError() == ERROR_FILE_NOT_FOUND,
>> +       "GetLastError() returns %ld\n", GetLastError());
>>      ok(test.cBytes == 0xA5,
>>         "LZOpenFileA set test.cBytes to %d\n", test.cBytes);
>>      ok(test.nErrCode == ERROR_FILE_NOT_FOUND,
>>
> 
> GetLastError returns a DWORD, so these formats need to be %d, not %ld.
> There is already a big effort to clean these up, so we don't need to
> introduce more.
Yes, but lz32 isn't converted yet so he is correct. I didn't do lz32 as
he was working on it and I didn't want to introduce conflicts.

bye
	michael
-- 
Michael Stefaniuc               Tel.: +49-711-96437-199
Sr. Network Engineer            Fax.: +49-711-96437-111
Red Hat GmbH                    Email: mstefani at redhat.com
Hauptstaetterstr. 58            http://www.redhat.de/
D-70178 Stuttgart



More information about the wine-devel mailing list