14 Aug
2009
14 Aug
'09
11:05 a.m.
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