r287008 - Add comments on how to print error retuned by tooling::Replacements::add().

Eric Liu via cfe-commits cfe-commits at lists.llvm.org
Tue Nov 15 11:05:41 PST 2016


Author: ioeric
Date: Tue Nov 15 13:05:40 2016
New Revision: 287008

URL: http://llvm.org/viewvc/llvm-project?rev=287008&view=rev
Log:
Add comments on how to print error retuned by tooling::Replacements::add().

Modified:
    cfe/trunk/include/clang/Tooling/Core/Replacement.h

Modified: cfe/trunk/include/clang/Tooling/Core/Replacement.h
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Tooling/Core/Replacement.h?rev=287008&r1=287007&r2=287008&view=diff
==============================================================================
--- cfe/trunk/include/clang/Tooling/Core/Replacement.h (original)
+++ cfe/trunk/include/clang/Tooling/Core/Replacement.h Tue Nov 15 13:05:40 2016
@@ -164,8 +164,9 @@ class Replacements {
   /// it returns an llvm::Error, i.e. there is a conflict between R and the
   /// existing replacements (i.e. they are order-dependent) or R's file path is
   /// different from the filepath of existing replacements. Callers must
-  /// explicitly check the Error returned. This prevents users from adding
-  /// order-dependent replacements. To control the order in which
+  /// explicitly check the Error returned, and the returned error can be
+  /// converted to a string message with `llvm::toString()`. This prevents users
+  /// from adding order-dependent replacements. To control the order in which
   /// order-dependent replacements are applied, use merge({R}) with R referring
   /// to the changed code after applying all existing replacements.
   /// Two replacements A and B are considered order-independent if applying them




More information about the cfe-commits mailing list