[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