diff options
author | MacroFake <falke.marco@gmail.com> | 2022-04-26 10:02:08 +0200 |
---|---|---|
committer | MacroFake <falke.marco@gmail.com> | 2022-04-26 10:01:54 +0200 |
commit | fa82a1ed833fd749849fa19267207b63e338d84d (patch) | |
tree | 12e9048b8872581feed595ffb76281754889d9d0 /test/lint | |
parent | 1ae65b4c5fbffb691e755dc0dacf1c63b55ad81b (diff) | |
download | bitcoin-fa82a1ed833fd749849fa19267207b63e338d84d.tar.xz |
lint: Mention NONFATAL_UNREACHABLE in lint-assertions.py
Diffstat (limited to 'test/lint')
-rwxr-xr-x | test/lint/lint-assertions.py | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/test/lint/lint-assertions.py b/test/lint/lint-assertions.py index a8d2b3927c..195ff33d11 100755 --- a/test/lint/lint-assertions.py +++ b/test/lint/lint-assertions.py @@ -30,20 +30,20 @@ def main(): r"[^_]assert\(.*(\+\+|\-\-|[^=!<>]=[^=!<>]).*\);", "--", "*.cpp", - "*.h" + "*.h", ], "Assertions should not have side effects:") - # Macro CHECK_NONFATAL(condition) should be used instead of assert for RPC code, where it - # is undesirable to crash the whole program. See: src/util/check.h + # Aborting the whole process is undesirable for RPC code. So nonfatal + # checks should be used over assert. See: src/util/check.h # src/rpc/server.cpp is excluded from this check since it's mostly meta-code. exit_code |= git_grep([ "-nE", - r"\<(A|a)ssert *\(.*\);", + r"\<(A|a)ss(ume|ert) *\(.*\);", "--", "src/rpc/", "src/wallet/rpc*", - ":(exclude)src/rpc/server.cpp" - ], "CHECK_NONFATAL(condition) should be used instead of assert for RPC code.") + ":(exclude)src/rpc/server.cpp", + ], "CHECK_NONFATAL(condition) or NONFATAL_UNREACHABLE should be used instead of assert for RPC code.") sys.exit(exit_code) |