From: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date: Mon, 10 Oct 2016 17:12:28 +0000 (+0100)
Subject: Enable some more compiler warnings
X-Git-Tag: dbus-1.11.8~46
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=f678f23be24006f6c3dccaff6c617086be275ca3;p=thirdparty%2Fdbus.git

Enable some more compiler warnings

Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=98192
---

diff --git a/configure.ac b/configure.ac
index d595ae0cb..a0833f0cb 100644
--- a/configure.ac
+++ b/configure.ac
@@ -1356,11 +1356,17 @@ TP_COMPILER_WARNINGS([WARNING_CFLAGS],
   [all \
    extra \
    char-subscripts \
+   implicit \
    missing-declarations \
    missing-prototypes \
    nested-externs \
    pointer-arith \
    pointer-sign \
+   old-style-declaration \
+   old-style-definition \
+   shadow \
+   strict-prototypes \
+   type-limits \
    cast-align \
    no-address \
    float-equal \
@@ -1375,15 +1381,9 @@ TP_COMPILER_WARNINGS([WARNING_CFLAGS],
   dnl   checking or not asserting
   dnl - missing field initializers being 0 is a C feature, not a bug
   dnl - unused-parameter is to make writing callbacks less annoying
-  dnl
-  dnl To be fixed one day:
-  dnl - pointer-sign is a workaround for fd.o #15522
-  dnl - type-limits is probably a bug too, but having the rest of -Wextra
-  dnl   is better than nothing
   [$DISABLE_UNUSED_WARNINGS \
    missing-field-initializers \
    unused-parameter \
-   type-limits \
   ])
 
 if test "x$GCC" = "xyes"; then