[PATCH 3 of 3] shell32: implement SHPathPrepareForWrite
Dmitry Timoshkov
dmitry at codeweavers.com
Sun Sep 2 20:59:48 CDT 2007
"Vincent Povirk" <madewokherd+d41d at gmail.com> wrote:
>> Why there is no check for SHCreateDirectoryExW return value?
> The return value of SHCreateDirectoryExW doesn't matter. All that
> matters is whether the directory exists when the function returns. It
> may or may not have existed when the function was called
> (SHCreateDirectoryExW fails when that happens, but
> SHPathPrepareForWrite should not fail).
>
> I could check the SHCreateDirectoryExW return value and probably
> decide on a result based on that, but I think it's better to write the
> result-deciding logic only once.
By checking return value of SHCreateDirectoryExW you can make a shortcut
in the deciding logic, in both cases: when the function fails you exit
right away (returning *correct* error to the caller and not a guessed one),
in the case of success GetFileAttributes call is redundant and you exit
right away again.
--
Dmitry.
More information about the wine-devel
mailing list