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: [8/9] multiple locations


Formerly, pending breakpoints were reported as:

(gdb) inf bre
Num Type           Disp Enb Address    What
1   breakpoint     keep y   <PENDING>  cos


now they're reported as:


(gdb) inf bre
Num     Type           Disp Enb  Address    What
1       breakpoint     keep y(p) <PENDING>  cos


This can break a front end which parses the old output, and this is indeed
the case for Emacs 22.1, which has already been released.

Does the (p) add anything useful to the user?  The manual says:

      An optional `(p)' suffix marks pending breakpoints

Is that not clear from the word PENDING?

If it's not needed can we please revert it to the old format?  If it is needed
can we find a format that doesn't break existing parsing?


-- 
Nick                                           http://www.inet.net.nz/~nickrob



breakpoint.c:

> > -  ui_out_field_fmt (uiout, "enabled", "%c", bpenables[(int) b->enable_state]);
> > -  ui_out_spaces (uiout, 2);
> > +  if (part_of_multiple)
> > +    ui_out_field_string (uiout, "enabled", 
> > +			 loc->shlib_disabled 
> > +			 ? (loc->enabled ? "y(p)" : "n(p)")
> > +			 : (loc->enabled ? "y" : "n"));


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