Requires CFLAGS='-std=gnu99' to compile on at least CentOS 5.11 about ntimed HOT 5 CLOSED

FYI, GCC 4.4.7 and Clang 3.4.2 on CentOS 6.5 don't suffer this issue.

from ntimed.

@joakimfors , I've created a patch in my repo, but I don't have an old enough Linux to test it with. Can you try it?

from ntimed.

make
cc -Wall -Werror   -c -o combine_delta.o combine_delta.c
combine_delta.c: In function ‘CD_AddSource’:
combine_delta.c:208: error: ‘INFINITY’ undeclared (first use in this function)
combine_delta.c:208: error: (Each undeclared identifier is reported only once
combine_delta.c:208: error: for each function it appears in.)
make: *** [combine_delta.o] Error 1

Perhaps the Makefile generation should be patched to include CFLAGS="-std=gnu99" instead?

from ntimed.

Now that's a worry. GNU decided that NAN was a GNU extension, but the link in my comment states clearly that INFINITY is part of the C99 standard and should exist everywhere.

Telling a portable program to compile with GNU extensions is not the right approach imo.

You don't by chance have default CFLAGS compiling for std89 or something do you?

Note that CentOS is from 2007.

from ntimed.

I have changed NAN which strictly speaking is only for float, to nan("") which is for double.

I'm not ever going to add a "-std=gnuanything" option if I can avoid it.

from ntimed.