laforge submitted this change.

View Change

Approvals: pespin: Looks good to me, but someone else must approve osmith: 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: I42f0ef1c9eea62a2512a15543ff45d776155fdc5
---
M configure.ac
1 file changed, 1 insertion(+), 0 deletions(-)

diff --git a/configure.ac b/configure.ac
index fce48b4..9454953 100644
--- a/configure.ac
+++ b/configure.ac
@@ -74,6 +74,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 29988. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: libsmpp34
Gerrit-Branch: master
Gerrit-Change-Id: I42f0ef1c9eea62a2512a15543ff45d776155fdc5
Gerrit-Change-Number: 29988
Gerrit-PatchSet: 2
Gerrit-Owner: laforge <laforge@osmocom.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: osmith <osmith@sysmocom.de>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-MessageType: merged