alexfh accepted this revision.
alexfh added a comment.
This revision is now accepted and ready to land.

LG with a couple of nits. Do you need someone to commit the patch for you?



================
Comment at: clang-tidy/modernize/RedundantVoidArgCheck.cpp:241
+    SourceLocation End, Begin;
+    auto SM = Result.SourceManager;
+    auto TypeLoc = Lambda->getLambdaClass()->getLambdaTypeInfo()->getTypeLoc();
----------------
s/auto /SourceManager */


================
Comment at: clang-tidy/modernize/RedundantVoidArgCheck.cpp:242
+    auto SM = Result.SourceManager;
+    auto TypeLoc = Lambda->getLambdaClass()->getLambdaTypeInfo()->getTypeLoc();
+    End = SM->getSpellingLoc(TypeLoc.getLocEnd());
----------------
Use the actual type name here as well.


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D49800



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

Reply via email to