This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [rfc] Shared libraries over the remote protocol, take two
- From: Eli Zaretskii <eliz at gnu dot org>
- To: Daniel Jacobowitz <drow at false dot org>
- Cc: gdb-patches at sourceware dot org, uweigand at de dot ibm dot com, pedro_alves at portugalmail dot pt
- Date: Mon, 18 Jun 2007 23:02:43 +0300
- Subject: Re: [rfc] Shared libraries over the remote protocol, take two
- References: <20070618150211.GA23415@caradoc.them.org>
- Reply-to: Eli Zaretskii <eliz at gnu dot org>
> Date: Mon, 18 Jun 2007 11:02:11 -0400
> From: Daniel Jacobowitz <drow@false.org>
> Cc: Ulrich Weigand <uweigand@de.ibm.com>, Pedro Alves <pedro_alves@portugalmail.pt>
>
> Index: gdb/NEWS
> ===================================================================
> --- gdb/NEWS.orig 2007-06-18 10:15:05.000000000 -0400
> +++ gdb/NEWS 2007-06-18 10:15:14.000000000 -0400
This is fine, thanks.
> --- gdb/doc/gdb.texinfo.orig 2007-06-18 10:15:05.000000000 -0400
> +++ gdb/doc/gdb.texinfo 2007-06-18 10:15:14.000000000 -0400
This is also fine, but...
> +The currently defined stop reasons are:
> +
> +@itemize @bullet
> +@item
> +If @var{n} is @samp{watch}, @samp{rwatch}, or @samp{awatch}, then the
> +packet indicates a watchpoint hit, and @var{r} is the data address, in
> +hex.
> +
> +@item
> +@cindex shared library events, remote reply
> +If @var{n} is @samp{library}, then the packet indicates that the
> +loaded libraries have changed. @value{GDBN} should use
> +@samp{qXfer:libraries:read} to fetch a new list of loaded libraries.
> +@var{r} is ignored.
> +@end itemize
I think using @itemize and conditional sentences like this is not the
very elegant and concise way of saying things. Compare with this
alternative:
The currently defined stop reasons are:
@table @samp
@item watch
@itemx rwatch
@itemx awatch
The packet indicates a watchpoint hit, and @var{r} is the data address, in
hex.
@cindex shared library events, remote reply
@item library
The packet indicates that the loaded libraries have changed.
@value{GDBN} should use @samp{qXfer:libraries:read} to fetch a new
list of loaded libraries. @var{r} is ignored.
@end table
I think this is better, as it makes the stop reasons stand out in a
more clear way. Do you agree?