PiotrZSL added inline comments.
================ Comment at: clang-tools-extra/docs/ReleaseNotes.rst:116-118 +- Improved :doc:`readability-identifier-naming + <clang-tidy/checks/readability/IdentifierNamingCheck>` check to allow for + Leading_upper_snake_case naming convention. ---------------- Wrong section, add it under "Changes in existing checks". Leading_upper_snake_case -> `Leading_upper_snake_case`. allow for -> support ================ Comment at: clang-tools-extra/docs/clang-tidy/checks/readability/identifier-naming.rst:2103 - - StructCase of ``lower_case`` + - StructCase of ``Leading_upper_snake_case`` - StructPrefix of ``pre_`` ---------------- do not change this, this (an changes in line ~2121) is for a lower_case scenario... Add some other example or explanation around line 33. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D158787/new/ https://reviews.llvm.org/D158787 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits