Re: [PATCH 2/5] Query supported notifications by qSupported

[Eli Zaretskii <eliz at gnu dot org>]

> Date: Mon, 11 Mar 2013 17:25:33 +0800
> From: Yao Qi <yao@codesourcery.com>
> CC: <gdb-patches@sourceware.org>
> 
> On 02/16/2013 05:32 PM, Eli Zaretskii wrote:
> > See above: I think you should show here the form of the
> > "notifications=" part in more detail.  In particular, the fact that
> > each notification is of the form NOTIFICATION.ANNEX1.ANNEX2... is
> > important and should be described and the meaning of the multiple
> > ANNEX parts explained.
> 
> How about this below,
> 
> .... supported by the
> stub.  Each notification in the string is composed by the name of the
> notification and the annexes, if any, which are separated by periods.
> 
> > 
> >> >+@item Notifications=@var{name}@r{[}.@var{annex}@r{]}@dots{}@r{[},@var{name}@r{[}.@var{annex}@r{]}@dots{}@r{]}@dots{}
> >> >+@cindex notifications, in remote protocol
> >> >+The remote stub supports a string of notifications.  @var{name} is
> >> >+the name of the notification and @var{annex} is the name of the annex,
> >> >+if the notification has the annex.
> > Here, too, we should say explicitly that there could be several
> > @var{annex} parts separated by periods.
> 
> and this,
> 
> if the notification has the annex.  Note that the notification may
> have several annexes, so here could be several @var{annex} parts
> separated by periods.

Thanks, I'm happy now.