[PATCH] D17852: Added formatAndApplyAllReplacements that works on multiple files in libTooling.

Daniel Jasper via cfe-commits cfe-commits at lists.llvm.org
Tue Mar 29 09:20:04 PDT 2016


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

Looks good.


================
Comment at: include/clang/Tooling/Refactoring.h:93
@@ +92,3 @@
+///
+/// See also "clang/Tooling/Core/Replacements.h".
+bool formatAndApplyAllReplacements(const Replacements &Replaces,
----------------
See also for what? Maybe just remove?

================
Comment at: include/clang/Tooling/Refactoring.h:95
@@ +94,3 @@
+bool formatAndApplyAllReplacements(const Replacements &Replaces,
+                                   Rewriter &Rewrite, StringRef Style);
+
----------------
Maybe Style should default to "file".


http://reviews.llvm.org/D17852





More information about the cfe-commits mailing list