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: Convenience functions


> Cc: gdb-patches@sourceware.org
> From: Tom Tromey <tromey@redhat.com>
> Date: Thu, 23 Oct 2008 13:13:15 -0600
> 
> I did write some docs.

Thanks.  I have two small requests for the documentation part:

> +Currently there is a single defined convenience function:

I would replace this with something more general, because we do hope
the list will grow.  Once somebody adds another function, this
sentence will become obsolete, but we might forget to change it.
OTOH, everybody can count to 1, so in its current form the sentence is
redundant.

> +@item $in_scope (@var{name})
> +Evaluate to @samp{1} if the symbol @var{name} is defined in the
> +program, or @samp{0} otherwise.

I suggest to change @var{name} to @var{symbol}, and then the text
could be a little bit more concise:

  Evaluate to @samp{1} if @var{symbol} is defined in the program, ...

By the way, is "defined in the program" accurate enough?  The name
"in_scope" does mean ``in scope'', right?  If so, I'd modify that
sentence to use ``in scope'' explicitly.  "Defined in the program"
could mean defined somewhere outside the scope of $pc.

Thanks.


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