krasimir updated this revision to Diff 96120.
krasimir added a comment.
- Introduce a proper abstraction
https://reviews.llvm.org/D32298
Files:
include/clang/Format/Format.h
lib/Format/Format.cpp
lib/Format/UnwrappedLineFormatter.cpp
lib/Format/UnwrappedLineFormatter.h
test/Format/inc
krasimir added inline comments.
Comment at: lib/Format/UnwrappedLineFormatter.cpp:845
+ if (!Invalid)
+os << " This might be due to a syntax error at line " << LineNumber
+ << ".";
djasper wrote:
> I wonder whether this might be confusing w
krasimir updated this revision to Diff 96115.
krasimir marked an inline comment as done.
krasimir added a comment.
- Removed double declaration
https://reviews.llvm.org/D32298
Files:
include/clang/Format/Format.h
lib/Format/Format.cpp
lib/Format/UnwrappedLineFormatter.cpp
lib/Format/Unw
krasimir updated this revision to Diff 96113.
krasimir added a comment.
- Address review comments
https://reviews.llvm.org/D32298
Files:
include/clang/Format/Format.h
lib/Format/Format.cpp
lib/Format/UnwrappedLineFormatter.cpp
lib/Format/UnwrappedLineFormatter.h
test/Format/incomplete
djasper added inline comments.
Comment at: include/clang/Format/Format.h:1527
+/// non-recoverable syntax error.
tooling::Replacements reformat(const FormatStyle &Style, StringRef Code,
ArrayRef Ranges,
This is a public interface
