laforge has submitted this change. (
https://gerrit.osmocom.org/c/osmo-hlr/+/29990 )
Change subject: Add -Werror=implicit-int -Werror=int-conversion
-Werror=old-style-definition
......................................................................
Add -Werror=implicit-int -Werror=int-conversion -Werror=old-style-definition
... if --enable-werror is used
Change-Id: Id465254bf12eb84acb116c86711a5b52b4d3ad35
---
M configure.ac
1 file changed, 1 insertion(+), 0 deletions(-)
Approvals:
Jenkins Builder: Verified
pespin: Looks good to me, approved
fixeria: Looks good to me, but someone else must approve
diff --git a/configure.ac b/configure.ac
index 8823288..78d0af9 100644
--- a/configure.ac
+++ b/configure.ac
@@ -93,6 +93,7 @@
if test x"$werror" = x"yes"
then
WERROR_FLAGS="-Werror"
+ WERROR_FLAGS+=" -Werror=implicit-int -Werror=int-conversion
-Werror=old-style-definition"
WERROR_FLAGS+=" -Wno-error=deprecated -Wno-error=deprecated-declarations"
WERROR_FLAGS+=" -Wno-error=cpp" # "#warning"
CFLAGS="$CFLAGS $WERROR_FLAGS"
--
To view, visit
https://gerrit.osmocom.org/c/osmo-hlr/+/29990
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-hlr
Gerrit-Branch: master
Gerrit-Change-Id: Id465254bf12eb84acb116c86711a5b52b4d3ad35
Gerrit-Change-Number: 29990
Gerrit-PatchSet: 3
Gerrit-Owner: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: merged