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.