Re: 2/2 start.exe: Handle the process title argument (try3)

[Dmitry Timoshkov]

"Alexander Nicolaysen Sørnes" <alex(a)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?

-- 
Dmitry.

2/2 start.exe: Handle the process title argument (try3)

[Alexander Nicolaysen Sørnes]

On Wednesday 30 January 2008 04:52:33 Dmitry Timoshkov wrote:
> "Alexander Nicolaysen Sørnes" <alex(a)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