[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