This revision was automatically updated to reflect the committed changes.
Closed by commit rL282253: [clang-format] support header deletion in
cleanupAroundReplacemnts. (authored by ioeric).
Changed prior to commit:
https://reviews.llvm.org/D24829?vs=72285&id=72286#toc
Repository:
rL LLVM
h
ioeric updated this revision to Diff 72285.
ioeric marked 2 inline comments as done.
ioeric added a comment.
- Addressed comments.
https://reviews.llvm.org/D24829
Files:
include/clang/Format/Format.h
lib/Format/Format.cpp
unittests/Format/CleanupTest.cpp
Index: unittests/Format/CleanupTe
djasper accepted this revision.
djasper added a comment.
This revision is now accepted and ready to land.
Looks good.
Comment at: lib/Format/Format.cpp:1557
@@ +1556,3 @@
+ const std::set HeadersToDelete) {
+ if (HeadersToDelete.find(HeaderName) != HeadersTo
ioeric created this revision.
ioeric added a reviewer: djasper.
ioeric added subscribers: klimek, cfe-commits.
- If a replacement has offset UINT_MAX, length 0, and a replacement text
that is an #include directive, this will insert the #include into the
correct block in the \p Code.
- If a rep
