[advpack/setupapi] LaunchINFSection

Dimi Paun dimi at lattica.com
Wed Nov 16 18:59:19 CST 2005


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 <dimi at lattica.com>
Lattica, Inc.




More information about the wine-devel mailing list