From bd3f3b553570d7a09cd16545c3d842d70bf3b7da Mon Sep 17 00:00:00 2001 From: Pedro Alves Date: Fri, 19 Apr 2013 18:32:54 +0000 Subject: [PATCH] Enable -Wpointer-sign by default. This enables -Wpointer-sign by default. I've checked that --enable-targets=all builds fine with the following as --host, on x86_64 Fedora 17 --build: x86_64 GNU/Linux i386 GNU/Linux i386 MinGW-w64 i386 msdos/djgpp OK? gdb/ 2013-04-19 Pedro Alves * configure.ac (build_warnings): Replace -Wno-pointer-sign with -Wpointer-sign. * configure: Regenerate. gdb/doc 2013-04-19 Pedro Alves * gdbint.texinfo (Misc Guidelines) : Replace -Wno-pointer-sign text with text on -Wpointer-sign. --- gdb/ChangeLog | 6 ++++++ gdb/configure | 2 +- gdb/configure.ac | 2 +- gdb/doc/ChangeLog | 5 +++++ gdb/doc/gdbint.texinfo | 13 ++++++------- 5 files changed, 19 insertions(+), 9 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index e795950d6c..5a2c3233da 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,9 @@ +2013-04-19 Pedro Alves + + * configure.ac (build_warnings): Replace -Wno-pointer-sign with + -Wpointer-sign. + * configure: Regenerate. + 2013-04-19 Pedro Alves * ser-tcp.c (net_read_prim): Cast second argument to recv to diff --git a/gdb/configure b/gdb/configure index f9d9a1745b..c8f3f1f55b 100755 --- a/gdb/configure +++ b/gdb/configure @@ -12452,7 +12452,7 @@ fi # NOTE: If you change this list, remember to update # gdb/doc/gdbint.texinfo. build_warnings="-Wall -Wdeclaration-after-statement -Wpointer-arith \ --Wformat-nonliteral -Wno-pointer-sign \ +-Wformat-nonliteral -Wpointer-sign \ -Wno-unused -Wunused-value -Wunused-function \ -Wno-switch -Wno-char-subscripts -Wmissing-prototypes \ -Wdeclaration-after-statement -Wempty-body" diff --git a/gdb/configure.ac b/gdb/configure.ac index ce6fa7a8a7..884b40ebb1 100644 --- a/gdb/configure.ac +++ b/gdb/configure.ac @@ -1925,7 +1925,7 @@ fi # NOTE: If you change this list, remember to update # gdb/doc/gdbint.texinfo. build_warnings="-Wall -Wdeclaration-after-statement -Wpointer-arith \ --Wformat-nonliteral -Wno-pointer-sign \ +-Wformat-nonliteral -Wpointer-sign \ -Wno-unused -Wunused-value -Wunused-function \ -Wno-switch -Wno-char-subscripts -Wmissing-prototypes \ -Wdeclaration-after-statement -Wempty-body" diff --git a/gdb/doc/ChangeLog b/gdb/doc/ChangeLog index bc7eaf4830..8d6cb96941 100644 --- a/gdb/doc/ChangeLog +++ b/gdb/doc/ChangeLog @@ -1,3 +1,8 @@ +2013-04-19 Pedro Alves + + * gdbint.texinfo (Misc Guidelines) : Replace + -Wno-pointer-sign text with text on -Wpointer-sign. + 2013-04-16 Tom Tromey * gdb.texinfo (Set Catchpoints): Mention earliest version of diff --git a/gdb/doc/gdbint.texinfo b/gdb/doc/gdbint.texinfo index 4a75c26c59..34e75e2094 100644 --- a/gdb/doc/gdbint.texinfo +++ b/gdb/doc/gdbint.texinfo @@ -6339,13 +6339,12 @@ Since @value{GDBN} uses the @code{format printf} attribute on all @code{printf} like functions this checks not just @code{printf} calls but also calls to functions such as @code{fprintf_unfiltered}. -@item -Wno-pointer-sign -In version 4.0, GCC began warning about pointer argument passing or -assignment even when the source and destination differed only in -signedness. However, most @value{GDBN} code doesn't distinguish -carefully between @code{char} and @code{unsigned char}. In early 2006 -the @value{GDBN} developers decided correcting these warnings wasn't -worth the time it would take. +@item -Wpointer-sign +This helps make sure @value{GDBN} code uses @code{gdb_byte} which is +really @code{unsigned char} for raw bytes instead of @code{char}, +whose signness is host-dependent. @sc{gcc} enables this with +@code{-Wall} since version 4.0. We enable it explicitly too to be +decoupled from future @sc{gcc} (or other compiler)'s defaults. @item -Wno-unused-parameter Due to the way that @value{GDBN} is implemented many functions have