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] Fixes to Cygwin-specific signal handling


> >         current_thread = th;
> >         if (!current_thread)
> >           thread_rec (thread_id, TRUE);
> 
> I've done this (with the last line as an assignment), but you'll
> have to help me find where the Coding Standard says that.
> 
> [1] mentions not using assignments inside if conditions, but I don't
> see any mention of ?:
> 
> [1] http://www.gnu.org/prep/standards/standards.html#Syntactic-Conventions

I think I got slightly confused by that one, as to what the "?"
applied to. But I guess that could be seen as a good reason for
avoiding it in this case :-P. Thanks for doing it anyways, even
though the reasons for it were not as strong as I thought.

-- 
Joel


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