From dd9aa0484e0b935fcf90d12337779e56a31f9352 Mon Sep 17 00:00:00 2001 From: Tom Tromey Date: Mon, 1 Jul 2013 19:52:32 +0000 Subject: [PATCH] add -Wold-style-definition This adds -Wold-style-definition to gdb's list of warnings. This found a couple of spots where "()" was used where "(void)" is more correct. Tested by rebuilding on x86-64 Fedora 18. * configure.ac (build_warnings): Add -Wold-style-definition. * configure: Rebuild. * machoread.c (_initialize_machoread): Use "(void)". * macrocmd.c (macro_inform_no_debuginfo): Fix formatting; use "(void)". --- gdb/ChangeLog | 8 ++++++++ gdb/configure | 2 +- gdb/configure.ac | 2 +- gdb/machoread.c | 2 +- gdb/macrocmd.c | 3 ++- 5 files changed, 13 insertions(+), 4 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 6e2d355..d3e76d3 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,5 +1,13 @@ 2013-07-01 Tom Tromey + * configure.ac (build_warnings): Add -Wold-style-definition. + * configure: Rebuild. + * machoread.c (_initialize_machoread): Use "(void)". + * macrocmd.c (macro_inform_no_debuginfo): Fix formatting; + use "(void)". + +2013-07-01 Tom Tromey + * configure.ac (build_warnings): Add -Wold-style-declaration. * configure: Rebuild. * dsrec.c (make_srec): Use "static const", not "const static". diff --git a/gdb/configure b/gdb/configure index 7e08fba..822f2df 100755 --- a/gdb/configure +++ b/gdb/configure @@ -12471,7 +12471,7 @@ build_warnings="-Wall -Wdeclaration-after-statement -Wpointer-arith \ -Wno-unused -Wunused-value -Wunused-function \ -Wno-switch -Wno-char-subscripts -Wmissing-prototypes \ -Wdeclaration-after-statement -Wempty-body -Wmissing-parameter-type \ --Wold-style-declaration" +-Wold-style-declaration -Wold-style-definition" # Enable -Wno-format by default when using gcc on mingw since many # GCC versions complain about %I64. diff --git a/gdb/configure.ac b/gdb/configure.ac index e0e5dd4..89a07b4 100644 --- a/gdb/configure.ac +++ b/gdb/configure.ac @@ -1941,7 +1941,7 @@ build_warnings="-Wall -Wdeclaration-after-statement -Wpointer-arith \ -Wno-unused -Wunused-value -Wunused-function \ -Wno-switch -Wno-char-subscripts -Wmissing-prototypes \ -Wdeclaration-after-statement -Wempty-body -Wmissing-parameter-type \ --Wold-style-declaration" +-Wold-style-declaration -Wold-style-definition" # Enable -Wno-format by default when using gcc on mingw since many # GCC versions complain about %I64. diff --git a/gdb/machoread.c b/gdb/machoread.c index 8d45f6f..b56e21c 100644 --- a/gdb/machoread.c +++ b/gdb/machoread.c @@ -1037,7 +1037,7 @@ static const struct sym_fns macho_sym_fns = { extern initialize_file_ftype _initialize_machoread; void -_initialize_machoread () +_initialize_machoread (void) { add_symtab_fns (&macho_sym_fns); diff --git a/gdb/macrocmd.c b/gdb/macrocmd.c index 250a1f0..b48c3c9 100644 --- a/gdb/macrocmd.c +++ b/gdb/macrocmd.c @@ -47,7 +47,8 @@ macro_command (char *arg, int from_tty) /* Prints an informational message regarding the lack of macro information. */ -static void macro_inform_no_debuginfo() +static void +macro_inform_no_debuginfo (void) { puts_filtered ("GDB has no preprocessor macro information for that code.\n"); } -- 2.7.4