Re: [patch] gdbserver: print inferior argv --debug

[Doug Evans <dje@google.com>]

Thanks.  I checked this in with the added double-quotes.

On Tue, Apr 20, 2010 at 12:53 AM, Pierre Muller
<pierre.muller@ics-cnrs.unistra.fr> wrote:
> Not an objection, rather a suggestion:
> what about wrapping these args into double quotes to
> see embedded spaces like this:
>> +     fprintf (stderr, "new_argv[%d] = \"%s\"\n", i, new_argv[i]);
>
> Pierre Muller
>
>> -----Message d'origine-----
>> De : gdb-patches-owner@sourceware.org [mailto:gdb-patches-
>> owner@sourceware.org] De la part de Doug Evans
>> Envoyé : Tuesday, April 20, 2010 9:21 AM
>> À : gdb-patches@sourceware.org
>> Objet : [patch] gdbserver: print inferior argv --debug
>>
>> Hi.
>>
>> I found this helpful debugging gdbserver --wrapper.
>>
>> I will check it in in a few days if there are no objections.
>>
>> 2010-04-20  Doug Evans  <dje@google.com>
>>
>>       * server.c (start_inferior): Print inferior argv if --debug.
>>
>> Index: server.c
>> ===================================================================
>> RCS file: /cvs/src/src/gdb/gdbserver/server.c,v
>> retrieving revision 1.116
>> diff -u -p -r1.116 server.c
>> --- server.c  16 Apr 2010 07:49:36 -0000      1.116
>> +++ server.c  20 Apr 2010 07:14:54 -0000
>> @@ -241,6 +241,14 @@ start_inferior (char **argv)
>>        new_argv[count] = NULL;
>>      }
>>
>> +  if (debug_threads)
>> +    {
>> +      int i;
>> +      for (i = 0; new_argv[i]; ++i)
>> +     fprintf (stderr, "new_argv[%d] = %s\n", i, new_argv[i]);
>> +      fflush (stderr);
>> +    }
>> +
>>  #ifdef SIGTTOU
>>    signal (SIGTTOU, SIG_DFL);
>>    signal (SIGTTIN, SIG_DFL);
>
>