Re: 2 CVS bugs

[Larry Jones]

Derek R. Price writes:
> 
> Whoops, just noticed you sent me the patch for getdate.c & not getdate.y.  So

Actually, they were both there.

> I researched the yyparse() prototype a little farther.  Bison (GNU yacc) 
> appears
> to generate prototypes for yyparse() automatically.  Apparently it's not 
> anything
> a user should override because it ifdef's on some things like YYPARSE_PARAM &
> __cplusplus for various reasons.

Note that it's also conditional on __GNUC__, so it's not generally
useful.  I endorse the suggested change -- declaring it without a
prototype finesses the question of the number and types of arguments
(which is what YYPARSE_PARAM is dealing with) but avoids the implicit
declaration (which more and more compilers are starting to warn about,
particularly since it's invalid in C99).  In my case, I was working with
the development source, but getdate.y was newer than getdate.c so make
rebuilt getdate.c using my version of bison which does not automatically
generate a declaration (and I don't know of any version of yacc that
generates a declaration).

-Larry Jones

I won't eat any cereal that doesn't turn the milk purple. -- Calvin