diff options
| author | Tom Tromey <tromey@adacore.com> | 2019-12-18 11:45:51 -0700 |
|---|---|---|
| committer | Tom Tromey <tromey@adacore.com> | 2019-12-18 11:50:13 -0700 |
| commit | 27e4fac77ea57b288ac1e08d936d9a8fdc01a1ee (patch) | |
| tree | 005a84cb132d183139e826e445bff2d2f535e4d9 /gdb/gdbsupport | |
| parent | e623f035020ea7baa64933b4afaaa564b4c7e26d (diff) | |
| download | gdb-27e4fac77ea57b288ac1e08d936d9a8fdc01a1ee.zip gdb-27e4fac77ea57b288ac1e08d936d9a8fdc01a1ee.tar.gz gdb-27e4fac77ea57b288ac1e08d936d9a8fdc01a1ee.tar.bz2 | |
Fix pthread_setname_np build error
My earlier patch to fix the pthread_setname_np build error on macOS
was incorrect. While the macOS man page claims that
pthread_setname_np returns void, in <pthread.h> it is actually
declared returning "int". I knew this earlier, but must have made
some mistake when preparing the patch for submission (perhaps when
removing the templates?).
This patch re-fixes the bug. I'm also applying it to the 9.1 branch.
Tested by building on macOS High Sierra.
gdb/ChangeLog
2019-12-18 Tom Tromey <tromey@adacore.com>
PR build/25268:
* gdbsupport/thread-pool.c (set_thread_name): Expect "int" return
type on macOS. Add comment.
Change-Id: Ib09da6ac33958a0d843f65df2a528112356e7de6
Diffstat (limited to 'gdb/gdbsupport')
| -rw-r--r-- | gdb/gdbsupport/thread-pool.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/gdb/gdbsupport/thread-pool.c b/gdb/gdbsupport/thread-pool.c index cd54895..a8c5fb7 100644 --- a/gdb/gdbsupport/thread-pool.c +++ b/gdb/gdbsupport/thread-pool.c @@ -49,8 +49,10 @@ set_thread_name (int (*set_name) (pthread_t, const char *), const char *name) set_name (pthread_self (), name); } +/* The macOS man page says that pthread_setname_np returns "void", but + the headers actually declare it returning "int". */ ATTRIBUTE_UNUSED static void -set_thread_name (void (*set_name) (const char *), const char *name) +set_thread_name (int (*set_name) (const char *), const char *name) { set_name (name); } |