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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
