[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 21:53:38 PDT 2020


Eugene.Zelenko added a comment.

In D86671#2246486 <https://reviews.llvm.org/D86671#2246486>, @dougpuob wrote:

> In D86671#2246480 <https://reviews.llvm.org/D86671#2246480>, @Eugene.Zelenko wrote:
>
>> By the word, you must mention new option in documentation too.
>
> Hi @Eugene.Zelenko,
> Is the `clang-tools-extra/docs/ReleaseNotes.rst` file that you mentioned?
>
> If YES, seems pretty simple with one line at the end for the new casing type, like:
>
>   Casing types include:
>   
>    - ``lower_case``,
>    - ``UPPER_CASE``,
>    - ``camelBack``,
>    - ``CamelCase``,
>    - ``camel_Snake_Back``,
>    - ``Camel_Snake_Case``,
>    - ``aNy_CasE``,
>    - ``szHungarianNotation``. 
>
> Is it right? and any more about document I have to do?

Yes, you are correct.


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