[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