From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <gdb-patches-return-16365-listarch-gdb-patches=sourceware.cygnus.com@sources.redhat.com>
Received: (qmail 25627 invoked by alias); 19 Jun 2002 05:42:05 -0000
Mailing-List: contact gdb-patches-help@sources.redhat.com; run by ezmlm
Precedence: bulk
List-Subscribe: <mailto:gdb-patches-subscribe@sources.redhat.com>
List-Archive: <http://sources.redhat.com/ml/gdb-patches/>
List-Post: <mailto:gdb-patches@sources.redhat.com>
List-Help: <mailto:gdb-patches-help@sources.redhat.com>, <http://sources.redhat.com/ml/#faqs>
Sender: gdb-patches-owner@sources.redhat.com
Received: (qmail 25530 invoked from network); 19 Jun 2002 05:41:59 -0000
Received: from unknown (HELO is.elta.co.il) (199.203.121.2)
  by sources.redhat.com with SMTP; 19 Jun 2002 05:41:59 -0000
Received: from is (is [199.203.121.2])
	by is.elta.co.il (8.9.3/8.8.8) with SMTP id IAA28032;
	Wed, 19 Jun 2002 08:38:59 +0300 (IDT)
Date: Tue, 18 Jun 2002 22:42:00 -0000
From: Eli Zaretskii <eliz@is.elta.co.il>
X-Sender: eliz@is
To: josef ezra <jezra@emc.com>
cc: Michael Snyder <msnyder@redhat.com>, gdb-patches@sources.redhat.com
Subject: Re: [RFA] new command: 'maintenance info lines' 
In-Reply-To: <022e01c21709$69a4aea0$ad219fa8@lss.emc.com>
Message-ID: <Pine.SUN.3.91.1020619083133.27660H-100000@is>
MIME-Version: 1.0
Content-Type: TEXT/PLAIN; charset=US-ASCII
X-SW-Source: 2002-06/txt/msg00359.txt.bz2


On Tue, 18 Jun 2002, josef ezra wrote:

> I'm adding this gdb.textinfo entry

Thanks.  A couple of minor comments:

> +This command takes same parameters as @code{disassemble}, but only 
> +dumps an 'address:line' pairs.

`address' and `line' stand for something else, so they should have the 
@var markup:

   ... @code{@var{address}:@var{line}} pairs ...

> +  Every pair means: "from this address 
> +to the next pair, the code was generated by that line".  

Please use `` and '' rather than " in Texinfo.  The former will look much 
better when typeset by TeX in the printed manual (in the Info output, 
they are converted to " automatically).

Other than that, the documentation patch is approved.