What the developer wrote
Authored by fanquake
tidy: remove deprecated header
```bash
15 | #warning The ClangTidyModuleRegistry.h header is deprecated and will be removed in LLVM 24. All of the symbols it used to define have been moved into ClangTidyModule.h.
| ^~~~~~~
[100%] Linking CXX shared module libbitcoin-tidy.so
```
What changed, and why it matters
This is a routine cleanup change in Bitcoin Core's internal developer tooling. It removes one outdated header file from a custom clang-tidy plugin used during code review and builds. There is no security relevance: no production code, consensus logic, wallet handling, or network behavior is affected.
No security action needed. Treat as normal maintenance/cleanup.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The commit removes #include <clang-tidy/ClangTidyModuleRegistry.h> from contrib/devtools/bitcoin-tidy/bitcoin-tidy.cpp because LLVM/clang-tidy deprecated that header and moved its symbols into ClangTidyModule.h. The diff is a single line deletion in a static-analysis helper plugin. It does not alter runtime behavior, parsing, validation, or any user-facing functionality.
Changed components
contrib/devtools/bitcoin-tidy/bitcoin-tidy.cppInspect captured patch +0 / −1
diff --git a/contrib/devtools/bitcoin-tidy/bitcoin-tidy.cpp b/contrib/devtools/bitcoin-tidy/bitcoin-tidy.cpp
index f2658b5a..bf79f387 100644
--- a/contrib/devtools/bitcoin-tidy/bitcoin-tidy.cpp
+++ b/contrib/devtools/bitcoin-tidy/bitcoin-tidy.cpp
@@ -5,7 +5,6 @@
#include "nontrivial-threadlocal.h"
#include <clang-tidy/ClangTidyModule.h>
-#include <clang-tidy/ClangTidyModuleRegistry.h>
class BitcoinModule final : public clang::tidy::ClangTidyModule
{
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.