[PATCH] D112409: [clang-tidy] Add check 'cert-err33-c'.
Aaron Ballman via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Oct 28 12:42:05 PDT 2021
aaron.ballman added a comment.
In D112409#3093815 <https://reviews.llvm.org/D112409#3093815>, @balazske wrote:
> Not sure if it is good to have such a test, the first and last function is enough?
Yeah, that's testing an awful lot. We don't usually aim for exhaustive tests with these configurable lists, so long as one or two entries are covered, we usually assume the rest will also be fine. Might be worth cutting the test down a bit.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D112409/new/
https://reviews.llvm.org/D112409
More information about the cfe-commits
mailing list