Re: Excplict document and check for C99 compiler (Was: Patch to allow compilation with old compilers (VS 2008))

[Paul Eggert]

On 9/18/18 2:00 PM, Simon Sobisch wrote:
> I also don't think it is worth the effort to check for every C99 item
> that is needed. But a minimal check like stdint.h, long long and
> declaration-after-code wouldn't hurt, would it?


'configure' is already doing all that.


> The patch is fine, but I don't think it provides a reason to not install
> my second patch for configure.ac [the place for the check is likely
> wrong, should be done after "try to find C99 compatibility-option"] and
> README.
>
> Do you see any specific issues with the proposed changes?

Yes, it's one more thing to maintain and it partly duplicates what 
'configure' is already doing. And the technical details are wrong, as 
Bison does not require a C99 compiler - it merely requires some C99 
features. Let's not bother with bikeshedding this; it's not worth the 
trouble.