[PATCH] D86671: [clang-tidy] Add new case type to check variables with Hungarian notation

Eugene Zelenko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Aug 27 21:56:04 PDT 2020


Eugene.Zelenko added a comment.

It'll be good idea to add test case.



================
Comment at: clang-tools-extra/docs/ReleaseNotes.rst:70
 
+* Added: Add IdentifierNamingCheck::CaseType, CT_HungarianNotation, supporting naming check with Hungarian notation.
+
----------------
Please move to `Changes in existing checks` section (in alphabetical order).

I think statement should describe user-facing changes.


================
Comment at: clang-tools-extra/docs/ReleaseNotes.rst:76
 
-- Improved :doc:`readability-identifier-naming
+- Improved: doc:`readability-identifier-naming
   <clang-tidy/checks/readability-identifier-naming>` check.
----------------
Unrelated change.


Repository:
  rCTE Clang Tools Extra

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

https://reviews.llvm.org/D86671



More information about the cfe-commits mailing list