[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