[prev in list] [next in list] [prev in thread] [next in thread] 

List:       wine-devel
Subject:    Re: 2/2 start.exe: Handle the process title argument (try3)
From:       Alexander Nicolaysen =?iso-8859-1?q?S=F8rnes?= <alex () thehandofagony ! com>
Date:       2008-01-30 12:43:37
Message-ID: 200801301343.37416.alex () thehandofagony ! com
[Download RAW message or body]

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



[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic