[PATCH] D32942: [clang-tidy] readability-function-size: add NestingThreshold param.
Malcolm Parsons via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Jun 14 06:11:08 PDT 2017
malcolm.parsons added a comment.
In https://reviews.llvm.org/D32942#780090, @lebedev.ri wrote:
> In https://reviews.llvm.org/D32942#780053, @malcolm.parsons wrote:
>
> > My prototype of this feature used `ifStmt(stmt().bind("if"), unless(hasParent(ifStmt(hasElse(equalsBoundNode("if"))))))`.
>
>
> The fix should ideally be fully contained in the `TraverseStmt()`, and should not be noticeably slower.
Yes, my prototype wasn't part of readability-function-size.
Repository:
rL LLVM
https://reviews.llvm.org/D32942
More information about the cfe-commits
mailing list