[OpenSIPS-Devel] [OpenSIPS/opensips] 810b27: Both true and false are now reserved words in a mo...
Răzvan Crainea
noreply at github.com
Thu Feb 20 12:43:31 UTC 2025
Branch: refs/heads/master
Home: https://github.com/OpenSIPS/opensips
Commit: 810b277afca859b3fd4b7e409809ded80a666885
https://github.com/OpenSIPS/opensips/commit/810b277afca859b3fd4b7e409809ded80a666885
Author: Peter Lemenkov <lemenkov at gmail.com>
Date: 2025-02-20 (Thu, 20 Feb 2025)
Changed paths:
M modules/statistics/statistics.c
Log Message:
-----------
Both true and false are now reserved words in a modern ANSI C
This prevents the following compile error:
```
Compiling statistics.c
gcc -fPIC -DPIC -O2 -flto=auto -ffat-lto-objects -fexceptions -g -grecord-gcc-switches -pipe -Wall -Wno-complain-wrong-lang -Werror=format-security -Wp,-U_FORTIFY_SOURCE,-D_FORTIFY_SOURCE=3 -Wp,-D_GLIBCXX_ASSERTIONS -specs=/usr/lib/rpm/redhat/redhat-hardened-cc1 -fstack-protector-strong -specs=/usr/lib/rpm/redhat/redhat-annobin-cc1 -m64 -march=x86-64 -mtune=generic -fasynchronous-unwind-tables -fstack-clash-protection -fcf-protection -mtls-dialect=gnu2 -fno-omit-frame-pointer -mno-omit-leaf-frame-pointer -fgnu89-inline -DMOD_NAME='statistics' -DPKG_MALLOC -DSHM_MMAP -DUSE_MCAST -DDISABLE_NAGLE -DSTATISTICS -DHAVE_RESOLV_RES -DF_MALLOC -DQ_MALLOC -DHP_MALLOC -DDBG_MALLOC -DHAVE_STDATOMIC -DHAVE_GENERICS -DNAME='"opensips"' -DVERSION='"3.5.4"' -DARCH='"x86_64"' -DOS='"linux"' -DCOMPILER='"gcc 15"' -D__CPU_x86_64 -D__OS_linux -D__SMP_yes -DCFG_DIR='"/etc/opensips/"' -DVERSIONTYPE='"git"' -DTHISREVISION='"5ebf81d1d"' -DFAST_LOCK -DADAPTIVE_WAIT -DADAPTIVE_WAIT_LOOPS=1024 -DHAVE_GETHOSTBYNAME2 -DHAVE_UNION_SEMUN -DHAVE_MSG_NOSIGNAL -DHAVE_MSGHDR_MSG_CONTROL -DHAVE_ALLOCA_H -DHAVE_TIMEGM -DHAVE_EPOLL -DHAVE_SIGIO_RT -DHAVE_SELECT -c stats_funcs.c -o stats_funcs.o
gcc -fPIC -DPIC -O2 -flto=auto -ffat-lto-objects -fexceptions -g -grecord-gcc-switches -pipe -Wall -Wno-complain-wrong-lang -Werror=format-security -Wp,-U_FORTIFY_SOURCE,-D_FORTIFY_SOURCE=3 -Wp,-D_GLIBCXX_ASSERTIONS -specs=/usr/lib/rpm/redhat/redhat-hardened-cc1 -fstack-protector-strong -specs=/usr/lib/rpm/redhat/redhat-annobin-cc1 -m64 -march=x86-64 -mtune=generic -fasynchronous-unwind-tables -fstack-clash-protection -fcf-protection -mtls-dialect=gnu2 -fno-omit-frame-pointer -mno-omit-leaf-frame-pointer -fgnu89-inline -DMOD_NAME='statistics' -DPKG_MALLOC -DSHM_MMAP -DUSE_MCAST -DDISABLE_NAGLE -DSTATISTICS -DHAVE_RESOLV_RES -DF_MALLOC -DQ_MALLOC -DHP_MALLOC -DDBG_MALLOC -DHAVE_STDATOMIC -DHAVE_GENERICS -DNAME='"opensips"' -DVERSION='"3.5.4"' -DARCH='"x86_64"' -DOS='"linux"' -DCOMPILER='"gcc 15"' -D__CPU_x86_64 -D__OS_linux -D__SMP_yes -DCFG_DIR='"/etc/opensips/"' -DVERSIONTYPE='"git"' -DTHISREVISION='"5ebf81d1d"' -DFAST_LOCK -DADAPTIVE_WAIT -DADAPTIVE_WAIT_LOOPS=1024 -DHAVE_GETHOSTBYNAME2 -DHAVE_UNION_SEMUN -DHAVE_MSG_NOSIGNAL -DHAVE_MSGHDR_MSG_CONTROL -DHAVE_ALLOCA_H -DHAVE_TIMEGM -DHAVE_EPOLL -DHAVE_SIGIO_RT -DHAVE_SELECT -c statistics.c -o statistics.o
statistics.c:101:31: error: expected identifier or ‘(’ before ‘true’
101 | unsigned long true;
| ^~~~
statistics.c:102:31: error: expected identifier or ‘(’ before ‘false’
102 | unsigned long false;
| ^~~~~
statistics.c:103:9: warning: no semicolon at end of struct or union
103 | } perc;
| ^
statistics.c: In function ‘reset_stat_series_slot’:
statistics.c:922:40: error: expected identifier before ‘true’
922 | ss->cache.perc.true -= slot->perc.true;
| ^~~~
statistics.c:923:40: error: expected identifier before ‘false’
923 | ss->cache.perc.false -= slot->perc.false;
| ^~~~~
statistics.c: In function ‘get_stat_series’:
statistics.c:987:48: error: expected identifier before ‘true’
987 | total = ss->cache.perc.true + ss->cache.perc.false;
| ^~~~
statistics.c:989:54: error: expected identifier before ‘true’
989 | ret = ss->cache.perc.true * ss->profile->factor / total;
| ^~~~
statistics.c: In function ‘update_stat_series’:
statistics.c:1075:41: error: expected identifier before ‘true’
1075 | s->perc.true += value;
| ^~~~
statistics.c:1076:48: error: expected identifier before ‘true’
1076 | ss->cache.perc.true += value;
| ^~~~
statistics.c:1078:41: error: expected identifier before ‘false’
1078 | s->perc.false -= value;
| ^~~~~
statistics.c:1079:48: error: expected identifier before ‘false’
1079 | ss->cache.perc.false -= value;
| ^~~~~
make[1]: Leaving directory '/builddir/build/BUILD/opensips-3.5.4-build/opensips-3.5.4/modules/statistics'
make[1]: *** [../../Makefile.rules:27: statistics.o] Error 1
make: *** [Makefile:198: modules] Error 2
```
Signed-off-by: Peter Lemenkov <lemenkov at gmail.com>
Commit: 495fe323c25c740e190043a4ba346914d27b5d9f
https://github.com/OpenSIPS/opensips/commit/495fe323c25c740e190043a4ba346914d27b5d9f
Author: Răzvan Crainea <razvan at opensips.org>
Date: 2025-02-20 (Thu, 20 Feb 2025)
Changed paths:
M modules/statistics/statistics.c
Log Message:
-----------
Merge pull request #3587 from lemenkov/bool_keywords
Both true and false are now reserved words in a modern ANSI C
Compare: https://github.com/OpenSIPS/opensips/compare/73ab11a31388...495fe323c25c
To unsubscribe from these emails, change your notification settings at https://github.com/OpenSIPS/opensips/settings/notifications
More information about the Devel
mailing list