clang-tidy: Disable `UndefinedBinaryOperatorResult` check in `src/ipc`
What the developer wrote
Authored by Hennadii Stepanov
clang-tidy: Disable `UndefinedBinaryOperatorResult` check in `src/ipc`
The warnings are false positive and have been fixed upstream.
See: https://github.com/capnproto/capnproto/pull/2334.
This change disables the `UndefinedBinaryOperatorResult` clang-tidy
check for source files generated by the `mpgen` tool.
What changed, and why it matters
This commit only changes build tooling configuration. It tells the clang-tidy static analyzer to ignore a specific warning class in generated IPC source files because the upstream Cap'n Proto project considers the warnings false positives. There is no change to Bitcoin Core's runtime code, no bug fix, and no security-relevant behavior change.
No security action needed. This is a build/CI linting configuration change.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The commit adds a .clang-tidy.in file under src/ipc that removes the clang-analyzer-core.UndefinedBinaryOperatorResult check, and wires it into the CMake build via configure_file. The affected files are generated by the mpgen tool from Cap’n Proto. The commit message explicitly states the warnings are false positives and references an upstream Cap’n Proto pull request (capnproto/capnproto#2334).
Changed components
src/ipc/.clang-tidy.insrc/ipc/CMakeLists.txtInspect captured patch +5 / −0
diff --git a/src/ipc/.clang-tidy.in b/src/ipc/.clang-tidy.in
new file mode 100644
index 00000000..2fc880ed
--- /dev/null
+++ b/src/ipc/.clang-tidy.in
@@ -0,0 +1,3 @@
+Checks: '
+-clang-analyzer-core.UndefinedBinaryOperatorResult,
+'
diff --git a/src/ipc/CMakeLists.txt b/src/ipc/CMakeLists.txt
index 904d72f5..8f0437be 100644
--- a/src/ipc/CMakeLists.txt
+++ b/src/ipc/CMakeLists.txt
@@ -21,3 +21,5 @@ target_link_libraries(bitcoin_ipc
core_interface
univalue
)
+
+configure_file(.clang-tidy.in .clang-tidy USE_SOURCE_PERMISSIONS COPYONLY)
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.