Age | Commit message (Collapse) | Author | Files | Lines | |
---|---|---|---|---|---|
2022-10-28 | Fixed main() prototypes in auto tests. | Andrew Clayton | 1 | -10/+10 | |
Future releases of GCC are planning to remove[0] default support for some old features that were removed from C99 but GCC still accepts. We can test for these changes by using the following -Werror= directives -Werror=implicit-int -Werror=implicit-function-declaration -Werror=int-conversion -Werror=strict-prototypes -Werror=old-style-definition Doing so revealed an issue with the auto/ tests in that the test programs always define main as int main() rather than int main(void) which results in a bunch of errors like build/autotest.c:3:23: error: function declaration isn't a prototype [-Werror=strict-prototypes] 3 | int main() { | ^~~~ build/autotest.c: In function 'main': build/autotest.c:3:23: error: old-style function definition [-Werror=old-style-definition] The fix was easy, it only required fixing the main prototype with find -type f -exec sed -i 's/int main() {/int main(void) {/g' {} \; Regardless of these upcoming GCC changes, this is probably a good thing to do anyway for correctness. [0]: https://fedoraproject.org/wiki/Changes/PortingToModernC Link: <https://lists.fedoraproject.org/archives/list/devel@lists.fedoraproject.org/message/CJXKTLXJUPZ4F2C2VQOTNMEA5JAUPMBD/> Link: <https://lists.fedoraproject.org/archives/list/devel@lists.fedoraproject.org/message/6SGHPHPAXKCVJ6PUZ57WVDQ5TDBVIRMF/> Reviewed-by: Alejandro Colomar <alx@nginx.com> Signed-off-by: Andrew Clayton <a.clayton@nginx.com> | |||||
2022-04-27 | Added NXT_MAYBE_UNUSED for __attribute__((__unused__)). | Alejandro Colomar | 1 | -0/+18 | |
When testing some configurations of compilers and OSes, I noticed that clang(1) 13 on Debian caused a function to be compiled but unused, and the compiler triggered a compile error. To avoid that error, use __attribute__((__unused__)). Let's call our wrapper NXT_MAYBE_UNUSED, since it describes itself more precisely than the GCC attribute name. It's also the name that C2x (likely C23) has given to the standard attribute, which is [[maybe_unused]], so it's also likely to be more readable because of that name being in ISO C. | |||||
2018-06-20 | Using own popcount where the compiler builtin is not available. | Sergey Kandaurov | 1 | -0/+13 | |
2018-04-11 | Configure scripts cleanup. | Valentin Bartenev | 1 | -6/+7 | |
Thanks to 洪志道 (Hong Zhi Dao). | |||||
2017-08-02 | Using old-style packed attribute specification for compatibility. | Max Romanov | 1 | -2/+3 | |
2017-07-18 | More accurate "packed" attribute declaration. | Valentin Bartenev | 1 | -0/+15 | |
2017-06-26 | Build on Solaris 11 fixed. | Max Romanov | 1 | -0/+13 | |