On Wednesday 30 January 2008 04:52:33 Dmitry Timoshkov wrote:
"Alexander Nicolaysen Sørnes" alex@thehandofagony.com wrote:
try 2 (thanks to Dmitry & Detlef)
- Use GlobalFree instead of LocalFree
- Fix indentation
- Remove part of a comment
try 3
- Use LocalFree, fixing CommandLineToargvW first
The following question from my original review still has not been answered:
wmain() already has arc/argv pair of parameters, why do you need to parse command line again?
We need to check for the first parameter beginning with a double quote ("). This is stripped from the argv array.
Alexander