summaryrefslogtreecommitdiffstats
path: root/buildtools
diff options
context:
space:
mode:
authorStefan Metzmacher <metze@samba.org>2014-11-07 09:36:16 +0100
committerJeremy Allison <jra@samba.org>2014-11-14 23:27:04 +0100
commit6ff9388172909ce249fd6254703eee707d821731 (patch)
tree42c5ba6ed805c1f0bd7490844e412a51f3683369 /buildtools
parent8ed899783d288c01f77f5139d3f5b6c26aa1fc7b (diff)
downloadsamba-6ff9388172909ce249fd6254703eee707d821731.tar.gz
samba-6ff9388172909ce249fd6254703eee707d821731.tar.xz
samba-6ff9388172909ce249fd6254703eee707d821731.zip
wafsamba: add -Werror=return-type for developer builds
This avoids errors like this: ../source3/utils/status.c: In function ‘print_share_mode’: ../source3/utils/status.c:126:3: error: ‘return’ with no value, in function returning non-void [-Werror=return-type] return; Signed-off-by: Stefan Metzmacher <metze@samba.org> Reviewed-by: Jeremy Allison <jra@samba.org>
Diffstat (limited to 'buildtools')
-rw-r--r--buildtools/wafsamba/samba_autoconf.py2
1 files changed, 2 insertions, 0 deletions
diff --git a/buildtools/wafsamba/samba_autoconf.py b/buildtools/wafsamba/samba_autoconf.py
index f60ce9dc2b..c193873116 100644
--- a/buildtools/wafsamba/samba_autoconf.py
+++ b/buildtools/wafsamba/samba_autoconf.py
@@ -671,6 +671,8 @@ def SAMBA_CONFIG_H(conf, path=None):
testflags=True)
conf.ADD_CFLAGS('-Werror=declaration-after-statement -Wdeclaration-after-statement',
testflags=True)
+ conf.ADD_CFLAGS('-Werror=return-type -Wreturn-type',
+ testflags=True)
conf.ADD_CFLAGS('-Wformat=2 -Wno-format-y2k', testflags=True)
# This check is because for ldb_search(), a NULL format string