[clang] [clang][analyzer] Move checker alpha.security.cert.pos.34c into security.PutenvWithAuto (PR #92424)
Balazs Benics via cfe-commits
cfe-commits at lists.llvm.org
Fri May 17 05:19:57 PDT 2024
steakhal wrote:
> The "cert" package looks not useful and the checker has not a meaningful name with the old naming scheme.
> Additionally tests and documentation is updated.
> The checker looks good enough to be moved into non-alpha package.
Personally, I prefer reviewing content changes separately from deciding/moving the checker out of alpha. Here are the reasons for doing so:
- When content is moved around, a different set of mistakes can be made than when updating some doc content.
- These are orthogonal decisions.
---
I'd suggest to split this PR into two:
- Keep this PR for updating the content.
- Have a separate PR for moving it out of alpha.
https://github.com/llvm/llvm-project/pull/92424
More information about the cfe-commits
mailing list