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

[Doug Evans <dje at google dot 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);
>
>