Hello,
It seems that we have been faced by the same problem and found the same solution. ;)
The comments in your patch make it sound like there are multiple values returned from Vista. It should not happen, hence the suggestion from Alexandre Julliard to check the compiler version.
However, my tests on the Wine Testbot tell me that Vista always returns DE_FILEDESTISFLD. I've not seen Vista return any other value. I suppose you didn't either. So we might be safe and assume that the original code is wrong or dated or that the Vista labeling is a mistake.
I can't explain how the original code could pass the testbot. Maybe it was added before the testbot existed, or when Vista had another Service Pack, or the testbot result was overlooked, I don't know.
Now Vista returns an unexpected error code and the test fails. The fix we have come up with passes successfully on all testbot VMs. It adds a new condition for the current Vista return value. The old condition remains because it is needed for W7U and W8. If the original code was valid for an older Vista Service Pack, the condition is still there to comply with the requirement of the older Vista.
I'll let Alexandre Julliard make up his mind with the provided information.
Keep up the good work!
Regards,
Le samedi 15 août 2015 à 09:05 +0800, Zhenbo Li a écrit :
Hello Olivier,
Thanks for your concern on my patch.
2015-08-15 4:34 GMT+08:00 Olivier F. R. Dierick o.dierick@piezo-forte.be:
I noticed tests failures in shlfileop while trying to add a new test. The failures were in parts that are not touched by my patches (same errors that you got). I have a patch ready for submission that correct the failures in the existing tests.
I've sent a patch long before[1] which I think is similar to patch 113659. Alexandre Julliard rejected my patch, and said to me "maybe a difference in the compiler version or something like that. I still would like to understand it instead of hiding it."
Can you find out what is the reason of this difference? Or find out how to convince Alexandre Julliard? lol