[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