From: Antonio Borneo Date: Sat, 15 May 2021 21:14:18 +0000 (+0200) Subject: Makefile: drop warning suppression on win build X-Git-Tag: v0.12.0-rc1~539 X-Git-Url: https://review.openocd.org/gitweb?p=openocd.git;a=commitdiff_plain;h=12e2dfd31ff07d9d13c7f66b799fad90a2698428 Makefile: drop warning suppression on win build Commit dcdf71c21b99 ("- fix signed/unsigned build errors under win32. Thanks Zach Welch ") in 2009 prevents gcc warnings on sign/unsigned comparisons while building for Win on folders 'helper' and 'server'. In 2011, commit b69119668ed8 ("RTOS Thread awareness support wip") uses the same method on the new folder 'rtos'. In mean time, all the incorrect sign/unsigned comparisons has been fixed and no warning is present with the default -Wextra flag that implies -Wsign-compare. The comment: # FD_* macros are sloppy with their signs on MinGW32 platform seems linked to some old implementation of MinGW32 include file that doesn't apply on current versions. Remove the obsolete hacks to suppress the warnings. Change-Id: I76dba9e54a647d3b9fbf1b7e9ae1844e3d7adc9a Signed-off-by: Antonio Borneo Reviewed-on: https://review.openocd.org/c/openocd/+/6254 Tested-by: jenkins Reviewed-by: Xiaofan Chen --- diff --git a/src/helper/Makefile.am b/src/helper/Makefile.am index 0d886843ce..c1aeebf000 100644 --- a/src/helper/Makefile.am +++ b/src/helper/Makefile.am @@ -30,12 +30,6 @@ noinst_LTLIBRARIES += %D%/libhelper.la %D%/jep106.inc \ %D%/jim-nvp.h -%C%_libhelper_la_CFLAGS = $(AM_CFLAGS) -if IS_MINGW -# FD_* macros are sloppy with their signs on MinGW32 platform -%C%_libhelper_la_CFLAGS += -Wno-sign-compare -endif - STARTUP_TCL_SRCS += %D%/startup.tcl EXTRA_DIST += \ %D%/bin2char.sh \ diff --git a/src/rtos/Makefile.am b/src/rtos/Makefile.am index 49cb830e5a..f09ac21a2e 100644 --- a/src/rtos/Makefile.am +++ b/src/rtos/Makefile.am @@ -31,10 +31,3 @@ noinst_LTLIBRARIES += %D%/librtos.la %D%/rtos_riot_stackings.h \ %D%/rtos_ucos_iii_stackings.h \ %D%/nuttx_header.h - -%C%_librtos_la_CFLAGS = $(AM_CFLAGS) - -if IS_MINGW -# FD_* macros are sloppy with their signs on MinGW32 platform -%C%_librtos_la_CFLAGS += -Wno-sign-compare -endif diff --git a/src/server/Makefile.am b/src/server/Makefile.am index 5f7469a849..fb5248bfde 100644 --- a/src/server/Makefile.am +++ b/src/server/Makefile.am @@ -14,10 +14,4 @@ noinst_LTLIBRARIES += %D%/libserver.la %D%/ipdbg.c \ %D%/ipdbg.h -%C%_libserver_la_CFLAGS = $(AM_CFLAGS) -if IS_MINGW -# FD_* macros are sloppy with their signs on MinGW32 platform -%C%_libserver_la_CFLAGS += -Wno-sign-compare -endif - STARTUP_TCL_SRCS += %D%/startup.tcl