[PATCH] D17704: Added interfaces for code-formatting while applying replacements.

Manuel Klimek via cfe-commits cfe-commits at lists.llvm.org
Mon Feb 29 03:43:39 PST 2016


klimek accepted this revision.
klimek added a comment.
This revision is now accepted and ready to land.

LG


================
Comment at: lib/Tooling/Core/Replacement.cpp:290
@@ +289,3 @@
+                                         const format::FormatStyle &Style) {
+  if (Replaces.empty()) return Replaces;
+
----------------
I'd return a default constructed Replacements() instead - I think that requires a bit less reasoning (as written currently, I'll need to reason that a copy is made to convince myself that this is correct).


http://reviews.llvm.org/D17704





More information about the cfe-commits mailing list