From mboxrd@z Thu Jan  1 00:00:00 1970
From: "Eli Zaretskii" <eliz@is.elta.co.il>
To: ac131313@cygnus.com
Cc: gdb-patches@sources.redhat.com
Subject: Re: [rfa/doc] Update ui-out; Was: [patch] Add col_name to ui_out_table_header()
Date: Fri, 06 Jul 2001 22:53:00 -0000
Message-id: <3405-Sat07Jul2001084846+0300-eliz@is.elta.co.il>
References: <Pine.SUN.3.91.1010705095541.15332N-100000@is> <3B4639A2.2070205@cygnus.com>
X-SW-Source: 2001-07/msg00171.html

> Date: Fri, 06 Jul 2001 18:20:18 -0400
> From: Andrew Cagney <ac131313@cygnus.com>
> 
> I think I got all of these.  Keep an eye out for the @pxref{} and 
> @footnote{}.

I did.  Here's a couple of minor comments.

> ! Most @code{ui_out} routines are of type @code{void}, The exceptions are

"The" should be "the".

> ! To recap: A @dfn{tuple} is a sequence of @dfn{fields}, each field
> ! containing information about an object; a @dfn{list} is a sequence of
> ! @dfn{fields} where each field describes an identical object.

The second "fields" should not be inside a @dfn, since we already have
one here.

> ! and correct implementation of the non-portable @footnote{The function
> ! cast is not portable ISO-C.} code sequence:

@footnote should immediately follow the letter after which you want
the footnote marker in the printed version, because otherwise TeX will
keep the blank between the footnote and the preceding text.  So:

  and correct implementation of the non-portable@footnote{The function