[clang] [analyzer] Avoid unnecessary super region invalidation in `CStringChecker` (PR #146212)
DonĂ¡t Nagy via cfe-commits
cfe-commits at lists.llvm.org
Wed Jul 2 07:47:21 PDT 2025
https://github.com/NagyDonat commented:
Overall I like this change, but I added a few suggestions in inline comments.
Also please follow the [Coding Standard](https://llvm.org/docs/CodingStandards.html#name-types-functions-variables-and-enumerators-properly) and use `UpperCamelCase` variables for variables (including ones that store lambda functions), while `lowerCamelCase` is for methods. Of course, it is also right to follow the "local customs" of older code fragments, but I'd suggest gradually transitioning to the global standard by following the coding standard for the freshly added variables and perhaps transitioning the variables which are mostly used in the code that you edit.
https://github.com/llvm/llvm-project/pull/146212
More information about the cfe-commits
mailing list