Eugene.Zelenko added inline comments.

================
Comment at: 
clang-tools-extra/docs/clang-tidy/checks/readability-identifier-naming.rst:2747
+Resolution order
+-------
+
----------------
Please make length same as title.


================
Comment at: 
clang-tools-extra/docs/clang-tidy/checks/readability-identifier-naming.rst:2762
+ - InlineNamespace ``[inline namespace]``
+ - Namespace ```[namespace]``
+ - <enum-members>
----------------
Extra back-tick.


================
Comment at: 
clang-tools-extra/docs/clang-tidy/checks/readability-identifier-naming.rst:2854
+Labels listed in ``<>`` brackets are semantic qualifiers and attempt to
+illustrate the semantic context within which clang-tidy resolves the
+classification. These are not formal semantic labels, rather labels which
----------------
`Clang-tidy`.


================
Comment at: 
clang-tools-extra/docs/clang-tidy/checks/readability-identifier-naming.rst:2857
+attempt disambiguation within the context of this document. For example,
+``<member-variables>`` identifiers that clang tidy is currently looking only at
+member variables.
----------------
Ditto.


================
Comment at: 
clang-tools-extra/docs/clang-tidy/checks/readability-identifier-naming.rst:2862
+that particular classification. These must be present in the declaration for
+clang-tidy to match the classification.
+
----------------
Ditto.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D126247

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to