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 fullname field for MI -break-info command


On Wed, Jan 25, 2006 at 04:31:19PM +0300, Vladimir Prus wrote:
> On Wednesday 25 January 2006 00:13, Daniel Jacobowitz wrote:
> 
> > > +        {
> > > +          struct symtab_and_line sal = find_pc_line (b->loc->address,
> > > 0); +          symtab_to_fullname (sal.symtab);
> > > +
> > > +          if (sal.symtab->fullname)
> > > +            {
> > > +              annotate_field(10);
> >
> >              annotate_field (10);
> 
> Is that a formatting change, or moving the "annotate_field" call out of "if"?

Sorry, I just meant the space before the parentheses.

> > But I don't think you need this at all; there's no point emitting
> > annotations when we're guarded by ui_out_is_mi_like_p.
> 
> For a future: can you explain what "annotate_field" does? The annotate.h file 
> has no comments at all, and gdbint has "annotate_field" only inside code 
> examples. And generally, what are "annotations"?

As Bob said, an old interface.  See "set annotate".

-- 
Daniel Jacobowitz
CodeSourcery


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