[PATCH] D91000: [clang-tidy] CERT MSC24-C Obsolescent Functions check
Aaron Ballman via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Jan 4 09:49:50 PST 2021
aaron.ballman added a comment.
In D91000#2469898 <https://reviews.llvm.org/D91000#2469898>, @lebedev.ri wrote:
> I think the question is, *why* are these checks being implemented?
> Just to claim that for some particular rule there is a check, and cross it off a list?
> Or for them to be actually used?
We have an existing CERT module for CERT-specific checks. When the check is honoring a published coding standard, that's usually sufficient justification -- but I do think the check should implement what the CERT recommendation says. If the CERT recommendation seems deficient in some way, it's reasonable to reach out to the authors (they allow comments on their wiki and are pretty good about responding to those comments) for clarification or to discuss potential improvements to the recommendation that would make the check more palatable.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D91000/new/
https://reviews.llvm.org/D91000
More information about the cfe-commits
mailing list