From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <gdb-patches-return-61744-listarch-gdb-patches=sources.redhat.com@sourceware.org>
Received: (qmail 16101 invoked by alias); 17 Feb 2009 20:28:42 -0000
Received: (qmail 15974 invoked by uid 22791); 17 Feb 2009 20:28:41 -0000
X-SWARE-Spam-Status: No, hits=-1.7 required=5.0 	tests=AWL,BAYES_00,SPF_SOFTFAIL
X-Spam-Check-By: sourceware.org
Received: from mtaout2.012.net.il (HELO mtaout2.012.net.il) (84.95.2.4)     by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Tue, 17 Feb 2009 20:28:34 +0000
Received: from conversion-daemon.i_mtaout2.012.net.il by i_mtaout2.012.net.il (HyperSendmail v2004.12) id <0KF800M008QGJK00@i_mtaout2.012.net.il> for gdb-patches@sources.redhat.com; Tue, 17 Feb 2009 22:29:02 +0200 (IST)
Received: from HOME-C4E4A596F7 ([84.228.82.14]) by i_mtaout2.012.net.il (HyperSendmail v2004.12) with ESMTPA id <0KF8009TH8WCG0O1@i_mtaout2.012.net.il>; Tue, 17 Feb 2009 22:29:01 +0200 (IST)
Date: Tue, 17 Feb 2009 20:30:00 -0000
From: Eli Zaretskii <eliz@gnu.org>
Subject: Re: MI solib notification
In-reply-to: <200902172244.48437.vladimir@codesourcery.com>
To: Vladimir Prus <vladimir@codesourcery.com>
Cc: drow@false.org, gdb-patches@sources.redhat.com, nickrob@snap.net.nz
Reply-to: Eli Zaretskii <eliz@gnu.org>
Message-id: <uy6w4vmdt.fsf@gnu.org>
References: <200901310010.46738.vladimir@codesourcery.com> <200902172208.37427.vladimir@codesourcery.com> <u4oysx3qq.fsf@gnu.org> <200902172244.48437.vladimir@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-02/txt/msg00363.txt.bz2

> From: Vladimir Prus <vladimir@codesourcery.com>
> Date: Tue, 17 Feb 2009 22:44:48 +0300
> Cc: drow@false.org,
>  gdb-patches@sources.redhat.com,
>  nickrob@snap.net.nz
> 
> How about the attached, instead?
> [...]
> +@item =library-loaded,...
> +Reports that a new library file was loaded by the program.  This
> +notification has 4 fields---@var{id}, @var{target-name},
> +@var{host-name}, and @var{symbols-loaded}. The @var{id} field is an
> +opaque identifier of the library.  For remote debugging case,
> +@var{target-name} and @var{host-name} fields give the name of the
> +library file on the target, and on the host respectively. For native
> +debugging, both those fields have the same value. The
> +@var{symbols-loaded} field reports if the debug symbols for this
> +library are loaded.

That's okay, but now I ask again why not do it like I suggested in the
first place, viz.:

    @item =library-loaded,@var{info}
    Reports that a new library file was loaded by the program.  @var{info}
    includes 4 fields:

    @table @code
    @item id="@var{id}"
    Opaque identifier of the library.
    @item target-name="@var{target-name}"
    @itemx host-name="@var{host-name}"
    For remote debugging case, @var{target-name} and @var{host-name}
    fields give the name of the library file on the target, and on the
    host respectively.  For native debugging, both those fields have the
    same value.
    ...

etc., you get the idea.  What you suggested now is very close to this,
but I think my suggestion makes it easier to read and grasp.

Also, there's one case in your text of only one space after a period
that ends a sentence.

Thanks.