[PATCH] shell32: Fix SHFileOperation(FO_MOVE) for creating subdirectories.
Zhenbo Li
litimetal at gmail.com
Sun Aug 30 20:29:15 CDT 2015
Hi Matteo,
Thanks for commenting on my patch
2015-08-27 0:45 GMT+08:00 Matteo Bruni <matteo.mystral at gmail.com>:
> SHCreateDirectoryExW can probably legitimately fail so I don't think
> an ERR is appropriate here. A WARN might be okay. While at it, please
> add a space between 'if' and '('.
>
I think it could be more challenging than I expected to handle the
errors. I'll look into it deeper.
> Unrelated preexisting issue, those helper functions for
> SHFileOperationW() like move_files() return a DWORD but the error code
> is otherwise int. It shouldn't matter in practice but it would be nice
> to fix that (in a separate patch).
I agree with you. DWORD is unsigned, but SHFileOperationW returns a
int value. I'll write a patch to fix it,
PS. Thanks to Sebastian Lackner's comment on my patch:
https://bugs.wine-staging.com/show_bug.cgi?id=541#c3
--
Have a nice day!
Zhenbo Li
More information about the wine-devel
mailing list