From: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date: Wed, 4 Mar 2015 10:41:47 +0000 (+0000)
Subject: Autotools: enable -Wsign-compare and optionally -Werror=sign-compare
X-Git-Tag: dbus-1.9.16~90
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=50bd76ac5b132f7033ac8b89d74675b5ce5e177f;p=thirdparty%2Fdbus.git

Autotools: enable -Wsign-compare and optionally -Werror=sign-compare

Bug: https://bugs.freedesktop.org/show_bug.cgi?id=17289
Reviewed-by: Ralf Habacker <ralf.habacker@freenet.de>
---

diff --git a/configure.ac b/configure.ac
index 5f78d1993..7443ae76c 100644
--- a/configure.ac
+++ b/configure.ac
@@ -1362,13 +1362,12 @@ TP_COMPILER_WARNINGS([WARNING_CFLAGS],
   dnl - unused-parameter is to make writing callbacks less annoying
   dnl
   dnl To be fixed one day:
-  dnl - sign-compare and pointer-sign are workarounds for fd.o #17433
+  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 \
-   sign-compare \
    pointer-sign \
    type-limits \
   ])