[gs-commits] mupdf 1.16.1.66 Enable signedness comparison warning.

[email protected] (Sebastian Rasmussen) Tue, 1 Oct 2019 14:55:52 +0000 (UTC)
Newsgroups gmane.comp.printing.ghostscript.cvs
Message-ID <[email protected]>
commit 13d1c2ad494216d3e7ff8545985cd8ee7afdab03
Author: Sebastian Rasmussen <[email protected]>
Date:   Fri Sep 27 18:35:58 2019 +0200

    Enable signedness comparison warning.
    
    Except for jbig2dec, freeglut and luratech everything builds
    with this warning now.

diff --git a/Makerules b/Makerules
index c0f8cb7..275be47 100644
--- a/Makerules
+++ b/Makerules
@@ -17,6 +17,8 @@ else
   LDREMOVEUNREACH := -Wl,--gc-sections
 endif
 
+CFLAGS += -Wsign-compare
+
 SANITIZE_FLAGS += -fsanitize=address
 SANITIZE_FLAGS += -fsanitize=leak
 
diff --git a/Makethird b/Makethird
index 68b4b69..bdc2ffb 100644
--- a/Makethird
+++ b/Makethird
@@ -279,7 +279,7 @@ LURATECH_CFLAGS += -Ithirdparty/luratech/lwf_jp2/library/source
 THIRD_CFLAGS += $(LURATECH_CFLAGS) -DHAVE_LURATECH
 
 $(OUT)/thirdparty/luratech/%.o: thirdparty/luratech/%.c
-	$(CC_CMD) -DLINUX $(LURATECH_CFLAGS) -Wno-tautological-compare -Wno-absolute-value
+	$(CC_CMD) -DLINUX $(LURATECH_CFLAGS) -Wno-tautological-compare -Wno-absolute-value -Wno-sign-compare
 
 else # HAVE_LURATECH
 
@@ -308,7 +308,7 @@ THIRD_SRC += thirdparty/jbig2dec/jbig2_text.c
 THIRD_CFLAGS += -Ithirdparty/jbig2dec
 
 $(OUT)/thirdparty/jbig2dec/%.o: thirdparty/jbig2dec/%.c
-	$(CC_CMD) -DHAVE_STDINT_H -DJBIG_EXTERNAL_MEMENTO_H=\"mupdf/memento.h\"
+	$(CC_CMD) -DHAVE_STDINT_H -DJBIG_EXTERNAL_MEMENTO_H=\"mupdf/memento.h\" -Wno-sign-compare
 
 endif
 
@@ -368,6 +368,7 @@ GLUT_OBJ := $(GLUT_SRC:%.c=$(OUT)/%.o)
 LOCAL_GLUT_CFLAGS += -Ithirdparty/freeglut/include
 LOCAL_GLUT_CFLAGS += -Ithirdparty/freeglut/src
 LOCAL_GLUT_CFLAGS += -DHAVE_UNISTD_H -DHAVE_STDINT_H -DHAVE_X11_EXTENSIONS_XRANDR_H
+LOCAL_GLUT_CFLAGS += -Wno-sign-compare
 
 GLUT_LIB := $(OUT)/libfreeglut.a
 

http://git.ghostscript.com/?p=mupdf.git;a=commit;h=13d1c2ad494216d3e7ff8545985cd8ee7afdab03

--
MuPDF library
Artifex Software, Inc.