[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
Sat Aug 29 20:42:50 PDT 2020
Eugene.Zelenko added a comment.
By the word, you must mention new option in documentation too.
================
Comment at: clang-tools-extra/docs/ReleaseNotes.rst:73
+- Added: Support variables could be checked with Hungarian Notation which the
+ prefix encodes the actual data type of the variable.
----------------
dougpuob wrote:
> Eugene.Zelenko wrote:
> > See examples for entry format below.
> Make it like the following ?
>
> ```
> Changes in existing checks
> ^^^^^^^^^^^^^^^^^^^^^^^^^^
>
> - Improved :doc:`readability-identifier-naming
> <clang-tidy/checks/readability-identifier-naming>` check.
>
> Added an option `GetConfigPerFile` to support including files which use
> different naming styles.
>
> - Improved :doc:`readability-identifier-naming
> <clang-tidy/checkers/readability-identifier-naming-hungarian-notation.cpp>` check.
>
> Support variables could be checked with Hungarian Notation which the prefix
> encodes the actual data type of the variable.
> ```
Yes, but link must be `clang-tidy/checks/readability-identifier-naming`, because it refer to documentation file, not regression test.
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