Re: bison bug.

[Akim Demaille]

>>>>> "Hans" == Hans Aberg <address@hidden> writes:

Hans> At 11:23 +0100 2001/12/30, Akim Demaille wrote:
>> ...Now bison has
>> 
>> if (nitems >= MAXSHORT) fatal (_("too many items (max %d)"),
>> MAXSHORT);

Hans> Under STD C, you should #include <limits.h>, and the code can
Hans> then read: if (nitems >= SHRT_MAX) fatal (_("too many items (max
Hans> %d)"), SHRT_MAX);

Hans> (And it can never happen that nitems > SHRT_MAX if nitems is a
Hans> short.)

No, this is wrong.  The limitation must not depend upon the host
running bison, but the least denominator with all the hosts on which
the parsers will be compiled.

Trust the code, people have been thinking about it.