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: [RFA] breakpoint.c, enable_command, missing break statement.


On Friday 04 March 2011 18:49:21, Michael Snyder wrote:
> I'm not entirely sure what to do with this one.
> 
> It clearly falls through, and it works as is, so the least change
> is just to comment it.

It looks irrelevant.  The intent of the code is obviously
to iterate over all breakpoints, or all kinds.  
The "continue" is continuing the loop hidden in
ALL_BREAKPOINTS at the next iteration.  But if you replace
all the continue's with break's it will still work the same,
because the "break" would break the the "switch", not the
loop.  I think if you do that the code ends up simpler to
read, with no magic.

-- 
Pedro Alves


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