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

Piotr Zegar via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Aug 25 09:36:49 PDT 2023


PiotrZSL added a comment.

Upload full path to avoid "Context not available."



================
Comment at: clang-tools-extra/docs/ReleaseNotes.rst:256
 
+- Improved :doc:`readability-identifier-naming
+  <clang-tidy/checks/readability/IdentifierNamingCheck>` Leading_upper_snake_case ->
----------------
put this entry in alphabetical order


================
Comment at: clang-tools-extra/docs/ReleaseNotes.rst:257
+- Improved :doc:`readability-identifier-naming
+  <clang-tidy/checks/readability/IdentifierNamingCheck>` Leading_upper_snake_case ->
+  Leading_upper_snake_case. allow for -> support
----------------
wrong name, such file does not exist


================
Comment at: clang-tools-extra/docs/ReleaseNotes.rst:257-258
+- Improved :doc:`readability-identifier-naming
+  <clang-tidy/checks/readability/IdentifierNamingCheck>` Leading_upper_snake_case ->
+  Leading_upper_snake_case. allow for -> support
+
----------------
PiotrZSL wrote:
> wrong name, such file does not exist
this message now says nothing... rephrase tis correctly... instead of just blindly copying text.


================
Comment at: clang-tools-extra/docs/clang-tidy/checks/readability/identifier-naming.rst:34
 
+Leading_upper_snake_case is a naming convention where the first word is capitalized
+followed by lower case word(s) seperated by underscore(s) '_'. Examples include:
----------------
put name into double `


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D158787/new/

https://reviews.llvm.org/D158787



More information about the cfe-commits mailing list