From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <gdb-patches-return-61982-listarch-gdb-patches=sources.redhat.com@sourceware.org>
Received: (qmail 11015 invoked by alias); 7 Mar 2009 10:29:56 -0000
Received: (qmail 11005 invoked by uid 22791); 7 Mar 2009 10:29:56 -0000
X-SWARE-Spam-Status: No, hits=1.5 required=5.0 	tests=AWL,BAYES_00,BOTNET,SPF_SOFTFAIL
X-Spam-Check-By: sourceware.org
Received: from mtaout1.012.net.il (HELO mtaout1.012.net.il) (84.95.2.1)     by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Sat, 07 Mar 2009 10:29:49 +0000
Received: from conversion-daemon.i-mtaout1.012.net.il by i-mtaout1.012.net.il (HyperSendmail v2007.08) id <0KG400A00T6CAC00@i-mtaout1.012.net.il> for gdb-patches@sourceware.org; Sat, 07 Mar 2009 12:30:25 +0200 (IST)
Received: from HOME-C4E4A596F7 ([77.127.192.247]) by i-mtaout1.012.net.il (HyperSendmail v2007.08) with ESMTPA id <0KG400ENQT6KKG80@i-mtaout1.012.net.il>; Sat, 07 Mar 2009 12:30:21 +0200 (IST)
Date: Sat, 07 Mar 2009 10:29:00 -0000
From: Eli Zaretskii <eliz@gnu.org>
Subject: Re: [remote/rfc] Let GDB know if a remote server originally attached to a process.
In-reply-to: <200903062124.02268.pedro@codesourcery.com>
To: Pedro Alves <pedro@codesourcery.com>
Cc: gdb-patches@sourceware.org, brobecker@adacore.com, drow@false.org
Reply-to: Eli Zaretskii <eliz@gnu.org>
Message-id: <ufxhpmxrd.fsf@gnu.org>
References: <200903040117.32166.pedro@codesourcery.com> <20090304191800.GB3729@adacore.com> <200903062124.02268.pedro@codesourcery.com>
X-IsSubscribed: yes
Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm
Precedence: bulk
List-Id: <gdb-patches.sourceware.org>
List-Subscribe: <mailto:gdb-patches-subscribe@sourceware.org>
List-Archive: <http://sourceware.org/ml/gdb-patches/>
List-Post: <mailto:gdb-patches@sourceware.org>
List-Help: <mailto:gdb-patches-help@sourceware.org>, <http://sourceware.org/ml/#faqs>
Sender: gdb-patches-owner@sourceware.org
X-SW-Source: 2009-03/txt/msg00092.txt.bz2

> From: Pedro Alves <pedro@codesourcery.com>
> Date: Fri, 6 Mar 2009 21:24:01 +0000
> Cc: Joel Brobecker <brobecker@adacore.com>,
>  Daniel Jacobowitz <drow@false.org>,
>  Eli Zaretskii <eliz@gnu.org>
> 
> Eli, is the documentation addition OK?

I have a few comments.

> +@item qAttached:@var{pid}
> +@cindex query attached, remote request
> +@cindex @samp{qAttached} packet
> +Return if the remote target is attached to the process with the
> +specified process ID, instead of having created it.

This sentence probably needs to be rephrased, but I don't understand
what it means, and so cannot suggest how to rephrase it.  It sounds
like you meant to say "Return an indication of whether the remote
target is attached ...", but then what is the purpose of adding
"instead of having created it"?  Could you please explain?

> @var{pid} is a
> +hexadecimal integer identifying the target process.

There's no such thing as "hexadecimal integer".  I think you mean
"an integer in hexadecimal format".

> +This query is used to, for example, know whether the remote process

Don't separate "to" from its verb:

  This query is used, for example, to know whether ...

Thanks.