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

Daniel Jasper via cfe-commits cfe-commits at lists.llvm.org
Mon Mar 21 06:48:29 PDT 2016


djasper added inline comments.

================
Comment at: include/clang/Tooling/Core/Replacement.h:231
@@ +230,3 @@
+typedef std::map<std::string, Replacements>
+    FileToReplacementsMap;
+
----------------
klimek wrote:
> Honestly, I'd get rid of the typedef. Daniel, what do you think?
I agree. I don't think it carries its weight and actually makes the code harder to read. Plus all users can probably use auto :-).


http://reviews.llvm.org/D17852





More information about the cfe-commits mailing list