[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