[PATCH] shell32: Fix SHFileOperation(FO_MOVE) for creating subdirectories.

Olivier F. R. Dierick o.dierick at piezo-forte.be
Sat Aug 15 00:39:16 CDT 2015


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 at 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
> 
> [1]: https://www.winehq.org/pipermail/wine-patches/2015-February/137382.html
> 
> 
-- 
        Olivier F. R. Dierick
        o.dierick at piezo-forte.be




More information about the wine-devel mailing list