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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to