On 12/5/2013 13:48, Zhenbo Li wrote:
> There are few lines in the code:
>
> + if (ret != ERROR_FILE_NOT_FOUND)
> + {
> + /* Vista would throw up a dialog box that we can't suppress */
>
> As this problem only exists in Vista && Windows 2008, so I think use
> GetVersion() is a better idea.
>
> ---
> dlls/shell32/tests/shlfileop.c | 13 ++++++++-----
> 1 file changed, 8 insertions(+), 5 deletions(-)
>
>
It's better as it is, just add a comment about win2008 if you like. The
rule is to skip basing on functionality instead of simple version check.