<div style="line-height:1.7;color:#000000;font-size:14px;font-family:arial"><div>I think that if you want GetLastError(),  "SetLastError( 0xdeadbeef );" should be added before every function which could  SetLastError().</div><div><br><div>--<br>ThanksŁ¬<div>Tao, Liu</div></div><div id="divNeteaseMailCard"></div><br><pre><br>At 2013-12-25 22:23:44,"Zhenbo Li" <litimetal@gmail.com> wrote:
>2013/12/25 Dmitry Timoshkov <dmitry@baikal.ru>:
>> Zhenbo Li <litimetal@gmail.com> wrote:
>>
>>> +    /* remove a nonexist directory */
>>> +    rc = RemoveDirectoryA(directory);
>>> +    todo_wine
>>> +    ok(!rc && GetLastError() == ERROR_FILE_NOT_FOUND,
>>> +        "Expected ERROR_FILE_NOT_FOUND, got ret=%d, gle=%d\n", rc, GetLastError() );
>>
>> This kind of test for several values simultaneously is really very awkward
>> as a starting point for any person who is going to work on this, or just is
>> looking for a simple bug to fix and greps for todo_wine in the tests. First
>> of all it's not clear where the failure is: is it the API failing to return
>> correct return value, or is it just last error which is set differently?
>> I admit that the error message mentions both values, but it's often more
>> useful to spot areas with todo_wine statements and clearly see the reason
>> of a failure without actually running the test.
>>
>> --
>> Dmitry.
>
>You are right, I'll send another patch.
>
>-- 
>Have a nice day!
>Zhenbo Li
>
>
</pre></div></div><br><br><span title="neteasefooter"><span id="netease_mail_footer"></span></span>