[PATCH] D24914: [clang-rename] Do not print out error message upon encountering multiple replacements in the same SourceLocation.

Kirill Bobyrev via cfe-commits cfe-commits at lists.llvm.org
Tue Sep 27 01:57:48 PDT 2016


omtcyfz updated this revision to Diff 72616.
omtcyfz added a comment.

Slightly change wording.


https://reviews.llvm.org/D24914

Files:
  clang-rename/RenamingAction.cpp

Index: clang-rename/RenamingAction.cpp
===================================================================
--- clang-rename/RenamingAction.cpp
+++ clang-rename/RenamingAction.cpp
@@ -68,12 +68,15 @@
                << ":" << FullLoc.getSpellingLineNumber() << ":"
                << FullLoc.getSpellingColumnNumber() << "\n";
       }
-      // FIXME: better error handling.
       tooling::Replacement Replace(SourceMgr, Loc, PrevNameLen, NewName);
       llvm::Error Err = FileToReplaces[Replace.getFilePath()].add(Replace);
+      // FIXME: An error might happen here if USRLocFinder finds the symbol
+      // twice or if the symbol happens to be in a header included multiple
+      // times independently. Such error doesn't mean clang-rename failure and
+      // therefore should be suppressed. However, more careful error handling
+      // might be better.
       if (Err)
-        llvm::errs() << "Renaming failed in " << Replace.getFilePath() << "! "
-                     << llvm::toString(std::move(Err)) << "\n";
+        llvm::consumeError(std::move(Err));
     }
   }
 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D24914.72616.patch
Type: text/x-patch
Size: 1087 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20160927/af2c59eb/attachment.bin>


More information about the cfe-commits mailing list