Re: [PATCH] Improves options in the manual

[Di-an JAN]

On Mon, 17 Nov 2008, Akim Demaille wrote:

> Le 17 nov. 08 à 01:34, Di-an JAN a écrit :
>
> > I think pine, my MUA, wrapped lines.  Sorry about that.
> > I'm resending the patch as an attachment.
> > I did add a long line, but texinfo doesn't seem to allow breaking
> > the @multitable line.
>
> I have plainly installed the patch and run make pdf, and the result shows 
> nothing about directives.  Here is the cross-options.texi I have.  There is 
> no % in there.

It's probably because I didn't test not building in the source directory.
I thought about it briefly and mixed up the source and build directories.
Here's a new patch, with comments on when special handling is needed.

 build-aux/cross-options.pl |   41 +++++++++++++++++++++++++++++++----------
 doc/bison.texinfo          |    9 ++++-----
 doc/local.mk               |    3 ++-
 src/getargs.c              |    5 +++++
 src/scan-gram.l            |    7 +++++++
 5 files changed, 49 insertions(+), 16 deletions(-)

2008-11-17  Di-an Jan  <address@hidden>

        Improves options in the manual.
        * doc/bison.texinfo (-g, -x): Add space before argument.
        (Option Cross Key): Implement FIXME: listing directives also.
        * build-aux/cross-options.pl:  Read from <STDIN> rather than <>.
        (Short Option): Special case -d.  Put arguments inside @option.
        (Bison Directive): Add column, automatically extracted from
        src/scan-gram.l (actual name passed as the first argument)
        with special case for %define.
        * doc/local.mk (doc/cross-options.texi): Pass src/scan-gram.l
        to build-aux/cross-options.pl.
        * src/getargs.c (usage): Document limitations of cross-options.pl.
        * src/scan-gram.l: Likewise.

cross-options.patch
Description: Text document