[PATCH 1/5] start.exe: unquote the title

[Damjan Jovanovic]

Signed-off-by: Damjan Jovanovic 
---
 programs/start/resources.h |  1 +
 programs/start/start.c     | 28 +++++++++++++++++++++++++++-
 programs/start/start.rc    |  2 ++
 3 files changed, 30 insertions(+), 1 deletion(-)

[Zebediah Figura]

Hello Damjan,

On 8/3/19 10:24 PM, Damjan Jovanovic wrote:
> int wmain (int argc, WCHAR *argv[])
>  {
>    SHELLEXECUTEINFOW sei;
> @@ -223,7 +242,13 @@ int wmain (int argc, WCHAR *argv[])
>    for (i=1; i<argc; i++) {
>                  /* parse first quoted argument as console title */
>                  if (!title && argv[i][0] == '"') {
> -
> 	title = argv[i];
> -
> 	/* it will remove at least 1 quote */
> -
> 	int maxChars = lstrlenW(argv[1]);
> -
> 	title = HeapAlloc(GetProcessHeap(), 0, maxChars*sizeof(WCHAR));
> -
> 	if (title)
> -
> 		parse_title(argv[i], title, maxChars);
> -
> 	else
> -
> 		fatal_string(STRING_NOMEMFORTITLE);
> continue;
> }
> if (argv[i][0] != '/')

Is this really salient enough to warrant error handling? I wouldn't 
particularly imagine we're likely to run out of memory here.

If so, perhaps we should use FormatMessage() instead of introducing a 
new string resource.

[Damjan Jovanovic]

On Sun, Aug 4, 2019 at 6:12 AM Zebediah Figura  wrote:
> Hello Damjan,

Hi
> On 8/3/19 10:24 PM, Damjan Jovanovic wrote:
> > int wmain (int argc, WCHAR *argv[])
> >  {
> >       SHELLEXECUTEINFOW sei;
> > @@ -223,7 +242,13 @@ int wmain (int argc, WCHAR *argv[])
> >       for (i=1; i<argc; i++) {
> >                  /* parse first quoted argument as console title */
> >                  if (!title && argv[i][0] == '"') {
> > -
> >                 title = argv[i];
> > -
> >                 /* it will remove at least 1 quote */
> > -
> >                 int maxChars = lstrlenW(argv[1]);
> > -
> >                 title = HeapAlloc(GetProcessHeap(), 0,
>
> maxChars*sizeof(WCHAR));
> > -
> >                 if (title)
> > -
> >                         parse_title(argv[i], title, maxChars);
> > -
> >                 else
> > -
> >                         fatal_string(STRING_NOMEMFORTITLE);
> >                 continue;
> >         }
> >         if (argv[i][0] != '/')
>
> Is this really salient enough to warrant error handling? I wouldn't
> particularly imagine we're likely to run out of memory here.

Is it now Wine policy to ignore memory allocation errors?
Since when?

[Alexandre Julliard]

Damjan Jovanovic  writes:
> On Sun, Aug 4, 2019 at 6:12 AM Zebediah Figura  wrote:
>
> Hello Damjan,
>
> Hi
>
> On 8/3/19 10:24 PM, Damjan Jovanovic wrote:
> > int wmain (int argc, WCHAR *argv[])
> >  {
> >       SHELLEXECUTEINFOW sei;
> > @@ -223,7 +242,13 @@ int wmain (int argc, WCHAR *argv[])
> >       for (i=1; i<argc; i++) {
> >                  /* parse first quoted argument as console title */
> >                  if (!title && argv[i][0] == '"') {
> > -
> >                 title = argv[i];
> > -
> >                 /* it will remove at least 1 quote */
> > -
> >                 int maxChars = lstrlenW(argv[1]);
> > -
> >                 title = HeapAlloc(GetProcessHeap(), 0, maxChars*sizeof(WCHAR));
> > -
> >                 if (title)
> > -
> >                         parse_title(argv[i], title, maxChars);
> > -
> >                 else
> > -
> >                         fatal_string(STRING_NOMEMFORTITLE);
> >                 continue;
> >         }
> >         if (argv[i][0] != '/')
>
> Is this really salient enough to warrant error handling? I wouldn't 
>  particularly imagine we're likely to run out of memory here.
>
> Is it now Wine policy to ignore memory allocation errors?
> Since when?

In a short-lived executable like start.exe, handling allocation errors
is usually a waste of time. If that offends you and you really want to
check, a simple abort will do. There's no need to add a translatable
resource string for this.