laforge submitted this change.

View Change

Approvals: Jenkins Builder: Verified fixeria: Looks good to me, approved
Add -Werror=implicit-int -Werror=int-conversion -Werror=old-style-definition

... if --enable-werror is used

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

diff --git a/configure.ac b/configure.ac
index 1604c88..017d78c 100644
--- a/configure.ac
+++ b/configure.ac
@@ -79,6 +79,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 29991. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: osmo-mgw
Gerrit-Branch: master
Gerrit-Change-Id: Ib397f3ad748e17f20b6177ef706af65088571f70
Gerrit-Change-Number: 29991
Gerrit-PatchSet: 2
Gerrit-Owner: laforge <laforge@osmocom.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-MessageType: merged