From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <gdb-patches-return-58461-listarch-gdb-patches=sources.redhat.com@sourceware.org>
Received: (qmail 9985 invoked by alias); 8 Sep 2008 19:25:31 -0000
Received: (qmail 9970 invoked by uid 22791); 8 Sep 2008 19:25:30 -0000
X-Spam-Check-By: sourceware.org
Received: from mtaout4.012.net.il (HELO mtaout4.012.net.il) (84.95.2.10)     by sourceware.org (qpsmtpd/0.31) with ESMTP; Mon, 08 Sep 2008 19:24:51 +0000
Received: from HOME-C4E4A596F7 ([84.229.211.50]) by i_mtaout4.012.net.il (HyperSendmail v2004.12) with ESMTPA id <0K6W007PT5XONY61@i_mtaout4.012.net.il> for gdb-patches@sourceware.org; Mon, 08 Sep 2008 22:25:01 +0300 (IDT)
Date: Mon, 08 Sep 2008 19:25:00 -0000
From: Eli Zaretskii <eliz@gnu.org>
Subject: Re: [rfc] [11/18] Cell multi-arch: Target description <compatible> infrastructure
In-reply-to: <200809081145.m88Bj171030625@d12av02.megacenter.de.ibm.com>
X-012-Sender: halo1@inter.net.il
To: Ulrich Weigand <uweigand@de.ibm.com>
Cc: gdb-patches@sourceware.org
Reply-to: Eli Zaretskii <eliz@gnu.org>
Message-id: <ur67ucug2.fsf@gnu.org>
References: <200809081145.m88Bj171030625@d12av02.megacenter.de.ibm.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: 2008-09/txt/msg00172.txt.bz2

> Date: Mon, 8 Sep 2008 13:45:01 +0200 (CEST)
> From: "Ulrich Weigand" <uweigand@de.ibm.com>
> Cc: gdb-patches@sourceware.org
> 
> Eli Zaretskii wrote:
> 
> > > +@var{arch} is an architecture name from the same selection
> > > +accepted by @code{set architecture} (@pxref{Targets, ,Specifying a
> > > +Debugging Target}).
> > 
> > I don't understand the purpose of "the same" here.  Same as what?
> 
> This element accepts the same set of architecture names as the set 
> accepted by the "set architecture" command.

Then I suggest to rephrase:

 +@var{arch} is one of the architectures from the set accepted by
 +@code{set architecture} (@pxref{Targets, ,Specifying a Debugging
 +Target}).

> The wording I've used is copied above is copied from the description
> of the <architecture> element

Please fix that one as well.

> See below for an updated patch.

Thanks, this is fine (with the above wording change).