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

Eric Liu via cfe-commits cfe-commits at lists.llvm.org
Tue Mar 29 08:11:52 PDT 2016


ioeric marked an inline comment as done.

================
Comment at: include/clang/Tooling/Refactoring.h:91
@@ +90,3 @@
+                                   Rewriter &Rewrite,
+                                   const format::FormatStyle &Style);
+
----------------
djasper wrote:
> Do you have a use case where we'd want to call this with a fixed style? Otherwise, I'd just remove this function for now.
If the .clang-format doesn't exist, it can only fall back to one default style, which is LLVM now. Shouldn't users have the flexibility to specify other styles if there is no .clang-format file?   


http://reviews.llvm.org/D17852





More information about the cfe-commits mailing list