laforge submitted this change.

View Change

Approvals: fixeria: Looks good to me, but someone else must approve pespin: Looks good to me, approved Jenkins Builder: Verified
Add -Werror=implicit-int -Werror=int-conversion -Werror=old-style-definition

... if --enable-werror is used

Change-Id: I8acd1b276f1ff8f015856d70fd582e457915ea11
---
M configure.ac
1 file changed, 1 insertion(+), 0 deletions(-)

diff --git a/configure.ac b/configure.ac
index 26a657e..63cd7a9 100644
--- a/configure.ac
+++ b/configure.ac
@@ -426,6 +426,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 change 29983. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I8acd1b276f1ff8f015856d70fd582e457915ea11
Gerrit-Change-Number: 29983
Gerrit-PatchSet: 6
Gerrit-Owner: laforge <laforge@osmocom.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-MessageType: merged