malcolm.parsons added inline comments.
================
Comment at: clang-tidy/readability/FunctionSizeCheck.cpp:38
+ } else {
+ assert(!isa<CompoundStmt>(Node));
+ if (TrackedParent.back())
----------------
I don't think this assert adds anything.
================
Comment at: clang-tidy/readability/FunctionSizeCheck.cpp:62
- if (TrackedParent.back())
+ // did we just process CompoundStmt? if yes, decrease current nesting
level.
+ if (isa<CompoundStmt>(Node))
----------------
Comments are sentences, so start with a capital letter.
Repository:
rL LLVM
https://reviews.llvm.org/D34202
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
