iwyu: Fix warning in `bench/pool.cpp`
What changed, and why it matters
This commit removes an unused header file (`<variant>`) from a benchmark source file. It is a code cleanup change with no security relevance. It does not affect the Bitcoin Core software that users run, only an internal performance-testing program.
No security action required. Treat as routine code hygiene.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The commit d92a20b310e2c024964e2a9ea8204e19f43a7d1a removes #include <variant> from src/bench/pool.cpp. The file is part of the benchmark suite, not production consensus or networking code. The change is described as an ‘iwyu’ (Include What You Use) warning fix and amends PR #35414. No functional code is modified.
Changed components
src/bench/pool.cppInspect captured patch +0 / −1
diff --git a/src/bench/pool.cpp b/src/bench/pool.cpp
index f5cfb99a..cf4ba132 100644
--- a/src/bench/pool.cpp
+++ b/src/bench/pool.cpp
@@ -10,7 +10,6 @@
#include <functional>
#include <unordered_map>
#include <utility>
-#include <variant>
template <typename Map>
void BenchFillClearMap(benchmark::Bench& bench, Map& map)
Why this scored 15/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.