[PATCH] D51949: [WIP][clang-tidy] initial ideas to isolate variable declarations

Jonas Toth via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Sep 11 16:06:02 PDT 2018


JonasToth added a comment.

Ty for the initial review, I am currently rewriting the approach for it.
I will probably make the business logic as standalone class/functions to
utilize this functionality in multiple checks.

I first try to evaluate my approach and as I had many unclear things i
opened this revision to point at actual code when I have a question.

Am 12.09.2018 um 00:59 schrieb Eugene Zelenko via Phabricator:

> Eugene.Zelenko added a comment.
> 
> I would suggest to use declaration or declarations in check name instead of abbreviation.
> 
> Actually there are existing coding guidelines for thus rule, like SEI CERT https://wiki.sei.cmu.edu/confluence/display/c/DCL04-C.+Do+not+declare+more+than+one+variable+per+declaration.
> 
> Repository:
> 
>   rCTE Clang Tools Extra
> 
> https://reviews.llvm.org/D51949


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D51949





More information about the cfe-commits mailing list