[doc] improve MI varobj introduction

Eli Zaretskii eliz@gnu.org
Fri Jan 5 09:26:00 GMT 2007


> From: Vladimir Prus <ghost@cs.msu.su>
> Date: Fri, 5 Jan 2007 11:39:09 +0300
> Cc: drow@false.org,
>  gdb-patches@sources.redhat.com
> 
> > > +Variable object is MI interface to work with expressions.
> > 
> > Perhaps it's an interface to work with named expressions, because I
> > believe you don't need anything to work with just expressions, do you?
> 
> Although you can use -data-evaluate-expression, using varobj is the recommended
> way. I don't think "named expressions" is the key here -- if MI was an interface
> in any object oriented language, you would not need varobj name at all. But since MI
> is pipe interface, you need some opaque token instead of object reference in a 
> programming language. So no fundamentally named expression are involved.
> How about:
> 
> 	Variable object is the recommended MI interface to work with expressions.

This doesn't give a clue why it is the recommended way.  I have
another suggestion, based on what you explained above:

  Variable objects are an MI convenience feature to reference
  expressions.  When a frontend creates a variable object, it
  specifies a name for an arbitrary expression in the debugged
  program.  That name can henceforth be used as an opaque handle for
  the expression.  The expression can be a simple variable, or it can
  be ...

Okay?

> > >                     Child variable objects can children themself,
> > > +util we reach leaf variable objects of built-in types. ^^^^^^^^
> >    ^^^^
> > Typos, and also something's wrong with this sentence in general.
> 
> Changed to:
> 
> 
> 	Child variable objects can themself have children,
> 	util we reach leaf variable objects of built-in types.

Hmmm... something is still wrong.  I think you meant this:

  A child variable object can itself have children, until we reach
  leaf variable objects which have built-in types.

> Revised patch attached -- hopefully better this time.

Yes, thanks.  The above two gotchas are the only things that need to
be taken care of.



More information about the Gdb-patches mailing list