[PATCH] D21601: Make tooling::applyAllReplacements return llvm::Expected<string> instead of empty string to indicate potential error.

Eric Liu via cfe-commits cfe-commits at lists.llvm.org
Fri Jun 24 05:45:06 PDT 2016


ioeric updated this revision to Diff 61780.
ioeric added a comment.

- fixed commenting.
- Make formatReplacemnts and cleanupAroundReplacements return llvm::Exppected<...>.


http://reviews.llvm.org/D21601

Files:
  include/clang/Format/Format.h
  include/clang/Tooling/Core/Replacement.h
  lib/Format/Format.cpp
  lib/Tooling/Core/Replacement.cpp
  lib/Tooling/Refactoring.cpp
  tools/clang-format/ClangFormat.cpp
  unittests/Format/CleanupTest.cpp
  unittests/Format/FormatTest.cpp
  unittests/Format/FormatTestJS.cpp
  unittests/Format/FormatTestJava.cpp
  unittests/Format/FormatTestProto.cpp
  unittests/Format/FormatTestSelective.cpp
  unittests/Format/SortImportsTestJS.cpp
  unittests/Format/SortIncludesTest.cpp
  unittests/Tooling/RefactoringTest.cpp
  unittests/Tooling/RewriterTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D21601.61780.patch
Type: text/x-patch
Size: 21792 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20160624/5c63f114/attachment-0001.bin>


More information about the cfe-commits mailing list