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

Reply via email to