Szelethus added inline comments.
================ Comment at: clang/docs/analyzer/checkers.rst:1935 + +alpha.security.cert.str.31c +""""""""""""""""""""""""""" ---------------- balazske wrote: > There are already more checkers that can check for CERT related problems but > not specially made for these. These checkers do not reside in this new `cert` > group. And generally a checker does not check for specifically a CERT rule, > instead for more of them or other things too, or more checkers can detect a > single rule. (And the user can think that only these CERT rules are checkable > that exist in this package, that is not true.) So I do not like the > introduction of this new `cert` package. (The documentation of existing > checkers lists if the checker is designed for a CERT rule.) I disagree to some extent. I think it would be great to have a `cert` package that houses all checkers for each of the rules with the addition of checker aliases. Clang-tidy has something similar as well! CHANGES SINCE LAST ACTION https://reviews.llvm.org/D70411/new/ https://reviews.llvm.org/D70411 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
