[PATCH] D126247: `readability-indentifier-naming` resolution order and examples

[Eugene Zelenko via Phabricator via cfe-commits]

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