[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