[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