Re: [PATCH] Autoload-breakpoints without report-async

[Eli Zaretskii <eliz at gnu dot org>]

> Date: Fri, 4 May 2012 09:00:30 +0800
> From: Hui Zhu <hui_zhu@mentor.com>
> CC: <gdb-patches@sourceware.org>, <Yao_Qi@mentor.com>, <stan_shebs@mentor.com>
> 
> Oops, I post the wrong version.  The attachment is the right version.  Please help me review it.
> +@item set breakpoint autoload query
> +If this option is @samp{query} (the default), @value{GDBN} will query to user
                                                                   ^^^^^^^^^^^^^
"query the user"

> +how to handle the autoload-breakpoints when @value{GDBN} connect to the stub.
                                                            ^^^^^^^
"connects"

> +@item set breakpoint autoload stub
> +If this option is @samp{stub}, the autoload-breakpoints of GDB will be removed
                                                              ^^^
"@value{GDBN}"

> +@item @samp{QBDP}@var{autoload-breakpoints base format}
> +@value{GDBN} use this packet to control the autoload-breakpoints in the remote.
                ^^^
"uses"

> +@table @samp
> +
> +@item @var{id}@samp{:}@var{command}@samp{:}@var{addr_string}@samp{:}@var{type}@samp{:}@var{ignore_num}

The whole table uses @samp as the default typeface, so you don't need
@samp in the @item line.  Every piece of text there that doesn't have
an explicit markup will be typeset as @samp automatically.

> +@item @var{id}
> +This is the id in hex string format of this command want to control.
> +this command wants to control.

 This is the id in hex string format of the breakpoint to which to
 apply this command.

> +                       @value{GDBN} will create this breakpoint as
> +it have been created in the target.  So when the target creates the
> +breakpoint, it needs to insert this breakpoint by itself.

  @value{GDBN} will assume the breakpoint already exists and is
  inserted in the remote, so the target needs to insert the breakpoint
  when it creates it.

> +When @value{GDBN} or the target get this packet, it will delete
> +the autoload-breakpoint @var{id}.                ^^^^^^^^^^^^^^

"they will delete"

OK with these changes.

Thanks.