shell32: fix program search in ShellExecuteEx, quote program name if it contains spaces (resend)

Juan Lang juan.lang at gmail.com
Fri Aug 14 11:05:23 CDT 2009


Hi again Stefan, I just took a little closer look, and you have more
whitespace issues in the patch:

-		    LPCWSTR p = space + 1;
+		    LPCWSTR p =  wszApplicationName + idx + 1;
Why two spaces after the =?

+        strcatW(wcmd, wQuote);
+    } else
     strcpyW(wcmd, wszApplicationName);
+
You should indent the strcpyW, even though it makes the patch a bit longer.

Finally, this isn't about whitespace but style:
+    if((needs_quote = NULL != strstrW(wszApplicationName,wSpace)))
+        len +=2;

The expression in the if is unnecessary complicated.  You could use
consistent spacing here too.
--Juan



More information about the wine-devel mailing list