This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] Add method/format information to =record-started


Simon Marchi <simon.marchi@ericsson.com> writes:

> Just to be clear, the inner if/else doesn't need them, but the outer one does.
>

No, it is not "GDB C Coding Standard" compliant, IMO.  In the quoted url
I gave,

"Any two or more lines in code should be wrapped in braces, even if they
are comments, as they look like separate statements:"

so...

> Here's the result:
>
>
>   if (started)
>     {
>       if (format != NULL)

brace is needed here...

> 	fprintf_unfiltered (
> 	  mi->event_channel,
> 	  "record-started,thread-group=\"i%d\",method=\"%s\",format=\"%s\"",
> 	  inferior->num, method, format);

and here

>       else
> 	fprintf_unfiltered (
> 	  mi->event_channel,
> 	  "record-started,thread-group=\"i%d\",method=\"%s\"",
> 	  inferior->num, method);
>     }
>   else
>     fprintf_unfiltered (mi->event_channel,
> 			"record-stopped,thread-group=\"i%d\"", inferior->num);

-- 
Yao (éå)


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]