Re: [PATCH/doc] Remove fixme of packet "k"

[Hui Zhu <hui_zhu@mentor.com>]

On 03/20/14 00:12, Eli Zaretskii wrote:
>> Date: Wed, 19 Mar 2014 16:02:29 +0800
>> From: Hui Zhu <hui_zhu@mentor.com>
>> CC: Eli Zaretskii <eliz@gnu.org>
>>
>> Hi Stan,
>>
>> According to your comments.  I make a new patch.
>>
>> Thanks,
>> Hui
>>
>> 2014-03-19  Stan Shebs  <stan@codesourcery.com>
>> 	    Hui Zhu  <hui@codesourcery.com>
>>
>> 	* gdb.texinfo (k packet): Remove fixme and update introduction.
>> 	(vKill packet): Add anchor.
>
> Thanks.  But these changes are both in the node called "Packets", not
> "k packet" or "vKill packet", so please change the text in parentheses
> to name the node (just once).
>
>> +A multiple-process target may choose to kill just one process, or all
>> +that that are under GDB's control.  For more precise control, use the
>     ^^^^^^^^^
> One "that" is redundant.
>
>> +vKill packet.(@pxref{vKill packet})
>
> @pxref should be before the period, like this:
>
>    vKill packet (@pxref{vKill packet}).
>
> OK with those changes.
>

Updated a new version according to your comments.
Please help me review it.

Thanks,
Hui

2014-03-20  Stan Shebs  <stan@codesourcery.com>
	    Hui Zhu  <hui@codesourcery.com>

	* gdb.texinfo (Packets): Remove fixme and update introduction of
	"k packet".  Add anchor to "vKill packet".

--- a/gdb/doc/gdb.texinfo
+++ b/gdb/doc/gdb.texinfo
@@ -33952,11 +33952,16 @@ step packet}.
  
  @item k
  @cindex @samp{k} packet
-Kill request.
+Kill the target process or processes.
  
-FIXME: @emph{There is no description of how to operate when a specific
-thread context has been selected (i.e.@: does 'k' kill only that
-thread?)}.
+The exact effect of this packet is not specified.  For a single-process
+target, it will kill that process if possible.
+
+A multiple-process target may choose to kill just one process, or all
+that are under GDB's control.  For more precise control, use the
+vKill packet (@pxref{vKill packet}).
+
+The @samp{k} packet has no reply.
  
  @item m @var{addr},@var{length}
  @cindex @samp{m} packet
@@ -34258,6 +34263,7 @@ request is completed.
  
  @item vKill;@var{pid}
  @cindex @samp{vKill} packet
+@anchor{vKill packet}
  Kill the process with the specified process ID.  @var{pid} is a
  hexadecimal integer identifying the process.  This packet is used in
  preference to @samp{k} when multiprocess protocol extensions are