[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