aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorfanquake <fanquake@gmail.com>2022-04-02 14:31:52 +0100
committerfanquake <fanquake@gmail.com>2022-04-02 14:34:39 +0100
commitd018db966ab8773863583f0283d57cd719accbaa (patch)
tree71b52607e1487a65ca47bef0a1aa5e40587b233c
parent03251b6b93fde57163cc32f6fa012241f3c6622d (diff)
parente8fc236da70085d30c90cdade06edfa1da855a6c (diff)
Merge bitcoin/bitcoin#24740: doc: remove incorrect mention of PR_GET_NAME
e8fc236da70085d30c90cdade06edfa1da855a6c refactor: add missing std:: includes to threadnames.cpp (fanquake) 87f3c04cc539c34d32af5fd59abef2c0b5faee26 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
-rw-r--r--src/util/threadnames.cpp4
1 files changed, 3 insertions, 1 deletions
diff --git a/src/util/threadnames.cpp b/src/util/threadnames.cpp
index 764fffabd7..a5a86d2598 100644
--- a/src/util/threadnames.cpp
+++ b/src/util/threadnames.cpp
@@ -6,7 +6,9 @@
#include <config/bitcoin-config.h>
#endif
+#include <string>
#include <thread>
+#include <utility>
#if (defined(__FreeBSD__) || defined(__OpenBSD__) || defined(__DragonFly__))
#include <pthread.h>
@@ -16,7 +18,7 @@
#include <util/threadnames.h>
#ifdef HAVE_SYS_PRCTL_H
-#include <sys/prctl.h> // For prctl, PR_SET_NAME, PR_GET_NAME
+#include <sys/prctl.h>
#endif
//! Set the thread's name at the process level. Does not affect the