diff options
author | fanquake <fanquake@gmail.com> | 2024-04-08 17:35:09 +0200 |
---|---|---|
committer | fanquake <fanquake@gmail.com> | 2024-04-08 17:35:15 +0200 |
commit | eaf186d1ee3a0271ec8cad65fa294b4298d79b0a (patch) | |
tree | a362cb2801ffdac4d39d4b719fafb8070899c203 /build-aux | |
parent | 3206e45412ded0e70c1f15ba66c2ba3b4426f27f (diff) | |
parent | 78407b99ed6dd17f687fcbfb0486ecc433302287 (diff) | |
download | bitcoin-eaf186d1ee3a0271ec8cad65fa294b4298d79b0a.tar.xz |
Merge bitcoin/bitcoin#29690: clang-tidy: Enable misc-no-recursion
78407b99ed6dd17f687fcbfb0486ecc433302287 [clang-tidy] Enable the misc-no-recursion check (dergoegge)
Pull request description:
Recursion is a frequent source of stack overflow bugs. Secondly, introduction of recursion can be non-obvious.
This PR proposes to use the clang-tidy `misc-no-recursion` check to make introduction of new recursion obvious. We don't make use of recursion a lot in our code base but there are a few places that need suppressions anyway (mostly the descriptor and univalue/rpc code).
ACKs for top commit:
stickies-v:
ACK 78407b99ed6dd17f687fcbfb0486ecc433302287
TheCharlatan:
Re-ACK 78407b99ed6dd17f687fcbfb0486ecc433302287
fanquake:
ACK 78407b99ed6dd17f687fcbfb0486ecc433302287
Tree-SHA512: 34126d704c46086fe7371906ca852c25ced1dbd5fcfd85bf623810cd171a797569a92a33c7e26b8dc01c30c7bbf81aa326718926e8354585091411989a4edb14
Diffstat (limited to 'build-aux')
0 files changed, 0 insertions, 0 deletions