[PATCH] D30777: Added `applyAtomicChanges` function.
Daniel Jasper via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Mar 22 11:54:50 PDT 2017
djasper accepted this revision.
djasper added a comment.
This revision is now accepted and ready to land.
Looks good.. Very nice :)
================
Comment at: lib/Tooling/Refactoring/AtomicChange.cpp:102
+ Code.substr(StartPos, EndPos - StartPos).split(Lines, '\n');
+ for (llvm::StringRef Line : Lines) {
+ if (Line.size() > ColumnLimit)
----------------
nit: remove braces (at least for consistency)
https://reviews.llvm.org/D30777
More information about the cfe-commits
mailing list