Skip to content

Commit d018db9

Browse files
committed
Merge bitcoin/bitcoin#24740: doc: remove incorrect mention of PR_GET_NAME
e8fc236 refactor: add missing std:: includes to threadnames.cpp (fanquake) 87f3c04 doc: remove incorrect mention of PR_GET_NAME (fanquake) Pull request description: By removing the whole comment. These `#include // For` comments are near impossible to maintain, pollute diffs, and generally don't add a lot of value. While here, also add the missing `std::` includes. ACKs for top commit: junderw: LGTM ACK e8fc236 Tree-SHA512: d29aff40c94f59c42f295a5738bc5ff2f4a2f2e6d270cc505f27d56d07d272597e2f8403d72fe45775661e1a1fc2af9fc52aeaeb41263bd3e9dfe255332383c8
2 parents 03251b6 + e8fc236 commit d018db9

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

src/util/threadnames.cpp

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,9 @@
66
#include <config/bitcoin-config.h>
77
#endif
88

9+
#include <string>
910
#include <thread>
11+
#include <utility>
1012

1113
#if (defined(__FreeBSD__) || defined(__OpenBSD__) || defined(__DragonFly__))
1214
#include <pthread.h>
@@ -16,7 +18,7 @@
1618
#include <util/threadnames.h>
1719

1820
#ifdef HAVE_SYS_PRCTL_H
19-
#include <sys/prctl.h> // For prctl, PR_SET_NAME, PR_GET_NAME
21+
#include <sys/prctl.h>
2022
#endif
2123

2224
//! Set the thread's name at the process level. Does not affect the

0 commit comments

Comments
 (0)