[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