On Wed, 2005-11-16 at 23:56 +0100, Raphael wrote:
> +    if (!bRet) {
> +      ERR("Failed to CopyFileW(%s, %s)\n", debugstr_w(src),
> debugstr_w(dst));
> +    }

You shouldn't issue ERR if the function can deal with
the error condition. Just return the appropriate error
code. ERR is to signal a case that should NEVER happen.
This is not the case in any of the instances where you
used ERR in this patch.

-- 
Dimi Paun <[EMAIL PROTECTED]>
Lattica, Inc.



Reply via email to