Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Migrate to modern C (C11 or C17) #27

Open
ysbaddaden opened this issue Jun 23, 2022 · 1 comment
Open

Migrate to modern C (C11 or C17) #27

ysbaddaden opened this issue Jun 23, 2022 · 1 comment

Comments

@ysbaddaden
Copy link
Owner

ysbaddaden commented Jun 23, 2022

C99 is nice but it's so damn old.

Migrating to C11 would allow to use platform agnostic atomics and maybe threads (must verify compatibility with #20). See https://en.cppreference.com/w/c/thread for details. C17 doesn't bring anything but fixed defects.

The library was also developed with clang 8 and GCC 7.5 which are so 2018. It would be nice to use more recent C compilers such as clang 14, GCC 13. They likely spout more useful warnings.

@ysbaddaden ysbaddaden changed the title Migrate to modern C (C11 or more) Migrate to modern C (C11 or C17) Jun 23, 2022
@ysbaddaden
Copy link
Owner Author

I installed clang-13 and gcc-11 and they didn't spout any additional warning. I also tested -c11 and -c17 instead of -c99 and it made no difference. It compiles properly and the test suite is passing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant