Stop suppressing -Wuninitialized in userland for WARNS >= 2 and <= 4.
authorSascha Wildner <saw@online.de>
Tue, 25 Feb 2014 23:03:57 +0000 (00:03 +0100)
committerSascha Wildner <saw@online.de>
Tue, 25 Feb 2014 23:09:08 +0000 (00:09 +0100)
This is from 2001 (via FreeBSD, who still have it). Since a lot of
important stuff is in that WARNS range (libc, most prominently) and
since the option has found some good bugs for us (just see the MFCs
of recent days), we definitely want it to be enabled. None of the
issues it reported turned out to be a false positive.

Do, however, turn off -Wmaybe-uninitialized in that WARNS range for
now.

Also, while here, turn on -Winit-self too.

share/mk/bsd.sys.mk

index 4faf438..a6fc792 100644 (file)
@@ -29,7 +29,7 @@ CWARNFLAGS    +=      -Werror
 .   endif
 .  endif
 .  if ${WARNS} >= 2
-CWARNFLAGS     +=      -Wall -Wformat-security -Wno-pointer-sign
+CWARNFLAGS     +=      -Wall -Wformat-security -Winit-self -Wno-pointer-sign
 .  endif
 .  if ${WARNS} >= 3
 CWARNFLAGS     +=      -Wextra -Wno-unused-parameter -Wstrict-prototypes\
@@ -45,9 +45,9 @@ CWARNFLAGS    +=      -Wchar-subscripts -Winline -Wnested-externs\
                        -Wredundant-decls
 .  endif
 .  if ${WARNS} >= 2 && ${WARNS} <= 4
-# XXX Delete -Wuninitialized by default for now -- the compiler doesn't
+# XXX Delete -Wmaybe-uninitialized by default for now -- the compiler doesn't
 # XXX always get it right.
-CWARNFLAGS     +=      -Wno-uninitialized
+CWARNFLAGS     +=      -Wno-maybe-uninitialized
 .  endif
 # Activate gcc47's -Wunused-but-set-variable (which is in -Wall) and
 # -Wunused-but-set-parameter (which is in -Wextra) only at WARNS >= 4