Re: [8/9] multiple locations

[Nick Roberts <nickrob@snap.net.nz>]

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"));