This revision was automatically updated to reflect the committed changes.
Closed by commit rL281457: Supports adding insertion around non-insertion
replacements. (authored by ioeric).
Changed prior to commit:
https://reviews.llvm.org/D24515?vs=71335&id=71337#toc
Repository:
rL LLVM
https://
ioeric updated this revision to Diff 71335.
ioeric added a comment.
- Updated comment.
https://reviews.llvm.org/D24515
Files:
include/clang/Tooling/Core/Replacement.h
lib/Tooling/Core/Replacement.cpp
unittests/Tooling/RefactoringTest.cpp
Index: unittests/Tooling/RefactoringTest.cpp
=
djasper accepted this revision.
djasper added a comment.
This revision is now accepted and ready to land.
Looks good.
Comment at: include/clang/Tooling/Core/Replacement.h:172
@@ -169,1 +171,3 @@
+ /// - aren't both inserts at the same code location.
+ /// Note: two insertion
ioeric updated this revision to Diff 71332.
ioeric marked an inline comment as done.
ioeric added a comment.
- Addressed comment.
https://reviews.llvm.org/D24515
Files:
include/clang/Tooling/Core/Replacement.h
lib/Tooling/Core/Replacement.cpp
unittests/Tooling/RefactoringTest.cpp
Index:
djasper added inline comments.
Comment at: include/clang/Tooling/Core/Replacement.h:172
@@ +171,3 @@
+ /// - They do not overlap.
+ /// - One replacement is insertion, and the other is a replacement with
+ /// length > 0, and the insertion is adjecent to but not contain
