From: Tom Tromey Date: Thu, 12 Dec 2019 13:09:40 +0000 (-0700) Subject: Fix the thread-pool.c compilation X-Git-Url: http://git.efficios.com/?p=deliverable%2Fbinutils-gdb.git;a=commitdiff_plain;h=2ffe5b9c792fe78dbbcbe31b6fea751285df8876 Fix the thread-pool.c compilation A recent commit removed DIAGNOSTIC_IGNORE_UNUSED_FUNCTION, which was used in thread-pool.c. This patch changes this code to use ATTRIBUTE_UNUSED instead. Tested by rebuilding. gdb/ChangeLog 2019-12-12 Tom Tromey * gdbsupport/thread-pool.c (set_thread_name): Use ATTRIBUTE_UNUSED. Change-Id: I56d46eaac73690565d0e52db1791411567a918dd --- diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 5ac4f9cb95..9dbc7b2012 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,8 @@ +2019-12-12 Tom Tromey + + * gdbsupport/thread-pool.c (set_thread_name): Use + ATTRIBUTE_UNUSED. + 2019-12-12 Luis Machado * gdbsupport/safe-strerror.c: Don't include diagnostics.h. diff --git a/gdb/gdbsupport/thread-pool.c b/gdb/gdbsupport/thread-pool.c index f6ea6d8b17..cd54895695 100644 --- a/gdb/gdbsupport/thread-pool.c +++ b/gdb/gdbsupport/thread-pool.c @@ -25,7 +25,6 @@ #include "gdbsupport/alt-stack.h" #include "gdbsupport/block-signals.h" #include -#include "diagnostics.h" /* On the off chance that we have the pthread library on a Windows host, but std::thread is not using it, avoid calling @@ -40,27 +39,22 @@ #include -DIAGNOSTIC_PUSH -DIAGNOSTIC_IGNORE_UNUSED_FUNCTION - /* Handle platform discrepancies in pthread_setname_np: macOS uses a single-argument form, while Linux uses a two-argument form. This wrapper handles the difference. */ -static void +ATTRIBUTE_UNUSED static void set_thread_name (int (*set_name) (pthread_t, const char *), const char *name) { set_name (pthread_self (), name); } -static void +ATTRIBUTE_UNUSED static void set_thread_name (void (*set_name) (const char *), const char *name) { set_name (name); } -DIAGNOSTIC_POP - #endif /* USE_PTHREAD_SETNAME_NP */ namespace gdb