Re: patch for gdb.texinfo

[Dmitry Sivachenko <dima at Chg dot RU>]

>
> > -intent is to aid the debugging of @dfn{dynamic arrays}, which cannot be
> > +intent is to aid the debugging of dynamic arrays, which cannot be
>
> Why is it a good idea to remove the @dfn here?

Because there is no definition of 'dynamic arrays' here in the text.

>
> > -@samp{-nx} option; @pxref{Mode Options, ,Choosing modes}.
> > +@option{-nx} option
> > +(@pxref{Mode Options, ,Choosing modes}).
> > +@comment pxref should be used only in () !!
>
> Yes, @pxref should be only in parens, so I think saying "see @ref"
> here (without the parens) is a better idea.

I agree.

>
> As for the replacements of @code and @samp with @env and @option:
> please note that this makes the GDB manual incompatible with all
> versions of Texinfo except the latest v4.0.  

That is true.

> Are you sure it's a good
> idea to break back-compatibility for no good reason (since they
> typeset in the same way)?

Do you use any new features of whatever OS you use?  This is also
incompatible with older versions...

I think we should use new features of texinfo.  May be in future versions
typeseting @env and @command  will differ from the current behaviour.
By the way, FreeBSD team already updated texinfo to 4.0 in base distribution.

--dima