diff --git a/gdb/ChangeLog b/gdb/ChangeLog index c80c5403516..5ac4f9cb95e 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,9 @@ +2019-12-12 Luis Machado + + * gdbsupport/safe-strerror.c: Don't include diagnostics.h. + (select_strerror_r): Use ATTRIBUTE_UNUSED instead of the diagnostics + macros. + 2019-12-11 Tom Tromey * tui/tui-win.c (tui_set_win_height_command): Call diff --git a/gdb/gdbsupport/safe-strerror.c b/gdb/gdbsupport/safe-strerror.c index 9973fa67857..a5ddf748353 100644 --- a/gdb/gdbsupport/safe-strerror.c +++ b/gdb/gdbsupport/safe-strerror.c @@ -18,7 +18,6 @@ along with this program. If not, see . */ #include "common-defs.h" -#include "diagnostics.h" #include /* There are two different versions of strerror_r; one is GNU-specific, the @@ -27,27 +26,20 @@ to solve this for us because IPA does not use Gnulib but uses this function. */ -/* We only ever use one of the two overloads, so suppress the warning for - an unused function. */ -DIAGNOSTIC_PUSH -DIAGNOSTIC_IGNORE_UNUSED_FUNCTION - /* Called if we have a XSI-compliant strerror_r. */ -static char * +ATTRIBUTE_UNUSED static char * select_strerror_r (int res, char *buf) { return res == 0 ? buf : nullptr; } /* Called if we have a GNU strerror_r. */ -static char * +ATTRIBUTE_UNUSED static char * select_strerror_r (char *res, char *) { return res; } -DIAGNOSTIC_POP - /* Implementation of safe_strerror as defined in common-utils.h. */ const char * diff --git a/include/ChangeLog b/include/ChangeLog index 52cdc0407ca..1444cc91461 100644 --- a/include/ChangeLog +++ b/include/ChangeLog @@ -1,3 +1,8 @@ +2019-12-12 Luis Machado + + * diagnostics.h (DIAGNOSTIC_IGNORE_UNUSED_FUNCTION). Remove + definitions. + 2019-12-11 Alan Modra * opcode/mmix.h (PUSHGO_INSN_BYTE): Make unsigned. diff --git a/include/diagnostics.h b/include/diagnostics.h index 2adaa4d3a03..019ade2567e 100644 --- a/include/diagnostics.h +++ b/include/diagnostics.h @@ -53,8 +53,6 @@ DIAGNOSTIC_IGNORE ("-Wdeprecated-declarations") # define DIAGNOSTIC_IGNORE_DEPRECATED_REGISTER \ DIAGNOSTIC_IGNORE ("-Wdeprecated-register") -# define DIAGNOSTIC_IGNORE_UNUSED_FUNCTION \ - DIAGNOSTIC_IGNORE ("-Wunused-function") # if __has_warning ("-Wenum-compare-switch") # define DIAGNOSTIC_IGNORE_SWITCH_DIFFERENT_ENUM_TYPES \ DIAGNOSTIC_IGNORE ("-Wenum-compare-switch") @@ -65,9 +63,6 @@ #elif defined (__GNUC__) /* GCC */ -# define DIAGNOSTIC_IGNORE_UNUSED_FUNCTION \ - DIAGNOSTIC_IGNORE ("-Wunused-function") - # define DIAGNOSTIC_IGNORE_STRINGOP_TRUNCATION \ DIAGNOSTIC_IGNORE ("-Wstringop-truncation") @@ -88,10 +83,6 @@ # define DIAGNOSTIC_IGNORE_DEPRECATED_REGISTER #endif -#ifndef DIAGNOSTIC_IGNORE_UNUSED_FUNCTION -# define DIAGNOSTIC_IGNORE_UNUSED_FUNCTION -#endif - #ifndef DIAGNOSTIC_IGNORE_SWITCH_DIFFERENT_ENUM_TYPES # define DIAGNOSTIC_IGNORE_SWITCH_DIFFERENT_ENUM_TYPES #endif