alexfh added inline comments.

================
Comment at: clang-tidy/misc/UnusedParametersCheck.cpp:180
+    // (constructor initializer counts for non-empty body).
+    if (StrictMode || llvm::distance(Function->getBody()->children()) > 0 ||
+        (isa<CXXConstructorDecl>(Function) &&
----------------
lebedev.ri wrote:
> So this checks that there are children.
Yes, and llvm::distance (recently renamed to llvm::size) doesn't work here any 
more, so the code now checks for iterator equality, which should be easier to 
read.


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D46951



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to