Re: texinfo-4.6.92 build feedback

Hi Nelson,

Thank you for all the reports.  A few followups below, I made the
obvious fixes for the others.

    static void
    mask_termsig (sigset_t *set)
    { ... }

    is seen by the compiler when sigset_t is not defined.

I added !SA_NOCLDSTOP to the test in signals.h which tries to deal with
this (apparently; this part of the code is pretty obscure to me).
That's what coreutils' sort.c does.  We'll see, I guess.

    session.c:4122: `strncasecmp' undeclared (first use in this function)

What system?  configure.ac does AC_REPLACE_FUNCS for strncasecmp, and
there is a lib/strncasecmp.c.  I surmise that the failing system somehow
detected that it had strncasecmp, yet the declaration was unavailable
when compiling info/session.c for some reason.

        ( * ( ( command ) ? ( command ) -> func : ( ( void * ) 0 ) ) ) ( active_window , count , 0 ) ;
    and the compiler rejects it with
            PGC-S-0098-Expression of type void * cannot be dereferenced (m-x.c: 151)
            PGC-S-0056-Attempt to call non-function (m-x.c: 151)

Hmm.  I'm not sure how to fix this.  The source line is:
  (*InfoFunction(command)) (active_window, count, 0);

I don't really want to avoid using the InfoFunction accessor macro,
which is all I can think of.  I added an explicit test for
InfoFunction(command) being non-null, but I'll be amazed if the compiler
is smart enough to then know we can't dereference NULL, as it looks like
the error is based on pure type analysis.

thanks,
karl
_______________________________________________
Texinfo home page: http://www.gnu.org/software/texinfo/
[email protected]
http://ff0.org/mailman/listinfo/texinfo-pretest