[PATCH] D158787: [clang-tidy][readability] add Leading_upper_snake_case

[Piotr Zegar via Phabricator via llvm-commits]

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