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

Jonas Toth via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Sep 13 13:24:02 PDT 2018


JonasToth added a comment.

Yes, do you think it should be included in the diag?

Am 13.09.2018 um 22:09 schrieb Roman Lebedev via Phabricator:

> lebedev.ri added inline comments.
> 
> ================
>  Comment at: clang-tidy/readability/IsolateDeclCheck.cpp:200
>  +
>  +  diag(WholeDecl->getBeginLoc(), "make only one declaration per statement")
>  +      << FixItHint::CreateReplacement(WholeDecl->getSourceRange(), Replacement);
> 
>  ----------------
> 
> I think you can get the count of declarations via `std::distance(WholeDecl->decl_begin(), WholeDecl->decl_end())`.
> 
> 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