[PATCH] D23193: [clang-rename] fix bug with initializer lists

Alexander Kornienko via cfe-commits cfe-commits at lists.llvm.org
Mon Aug 8 07:06:38 PDT 2016


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

LG with a nit.


================
Comment at: clang-rename/USRFinder.cpp:100
@@ +99,3 @@
+      if (const clang::FieldDecl *FieldDecl = Initializer->getMember()) {
+        const auto InitBeginLoc = Initializer->getSourceLocation();
+        const auto InitEndLoc = Lexer::getLocForEndOfToken(
----------------
It might be not clear from the context what `auto` means here, I'd use the explicit type.


https://reviews.llvm.org/D23193





More information about the cfe-commits mailing list