shell32/tests: Add test for GetLastError in shlfileop

Zhenbo Li litimetal at gmail.com
Tue Jan 28 07:26:55 CST 2014


2014-01-28 Dmitry Timoshkov <dmitry at baikal.ru>
>
>
> > There are many 'todo_wine' marks in shell32 test, and it really caused
> > bugs(like 34324).
>
> According to the comments and the attached hack in the bug 34324 the problem
> there is not related to the last error set but to the returned error value,
> so you probably should add the tests for that particular behaviour.
>
>
> > Maybe such test cases can contribute to further development?
>
> Adding the tests is good in general, but the tests should be added very
> carefully, and check something that real applications could depend upon.
> Adding blanket tests just for everything is a waste of time IMHO.

Thanks for checking that.
As MSDN[1] said, "Do not use GetLastError with the return values of this
(SHFileOperation) function."

I think wine's implement of SHFileOperation needs a big change.
Before hacking it, I hope there could be sufficient testcases to
help me understand that function.

I had to admit that I'm lack of experience, and I don't know
how to judge if a testcase is necessary.

Thank you very much.

[1]: http://msdn.microsoft.com/en-us/library/windows/desktop/bb762164%28v=vs.85%29.aspx



-- 
Have a nice day!
Zhenbo Li



More information about the wine-devel mailing list