AmigaLabs / clib4

Clib4 for AmigaOS4
Other
14 stars 7 forks source link

clib4 is broken for c90 (ANSI) compatibility #175

Closed 3246251196 closed 5 months ago

3246251196 commented 5 months ago

inline is not a known keyword for std ANSI. This means that project that build with std ANSI will not build.

3246251196 commented 5 months ago
./library/crypt/crypt_des.c
./library/include/ctype.h
./library/include/endian.h
./library/include/features.h
./library/include/fenv.h
./library/include/stdatomic.h
./library/include/sys/byteswap.h
./library/include/sys/queue.h
./library/include/time.h
./library/locale/dcngettext.c
./library/macros.h
./library/math/fenv.c
./library/math/invtrig.h
./library/math/k_logf.h
./library/math/k_log.h
./library/math/math_headers.h
./library/math/s_fabs.c
./library/math/s_fma.c
./library/math/s_nan.c
./library/math/s_nanf.c
./library/posix/memalign.c
./library/setjmp/sdt.h
./library/socket/getaddrinfo.c
./library/stdio/vfprintf.c
./library/stdlib/arc4random.c
./library/stdlib/qsort.c
./library/stdlib/qsort_r.c
./library/stdlib/strtod.c
./library/string/atomic_arch.h
./library/string/atomic.h
./library/string/memchr.c
./library/string/memcmp.c
./library/string/strcmp.c
./library/string/strlen.c
./library/time/strftime.c
./library/time/time_headers.h
./misc/PatchForSPE.cpp
./test_programs/math/libm_test/libm-bench.cpp
./test_programs/math/libm_test/libm-test.c
./test_programs/math/libm_test/test-double.c
./test_programs/math/libm_test/test-float.c
./test_programs/memory/memcpy/src/types.h
./test_programs/memory/memset/src/impl.cpp
./test_programs/memory/memset/src/types.h
./test_programs/memory/memtest.c
./test_programs/memory/ramspeed.c
./test_programs/memory/realloc3.cpp
./test_programs/misc/md5sum.c
afxgroup commented 5 months ago

I think only include files needs to be fixed because clib4 can't be compiled with -ansi (-std=c90) flag. All other files can use inline without any problem.

afxgroup commented 5 months ago

Fixed in latest commit