ilya-biryukov accepted this revision. ilya-biryukov added a comment. This revision is now accepted and ready to land.
LGTM with a minor naming suggestion. ================ Comment at: include/clang/Format/Format.h:20 #include "clang/Tooling/Core/Replacement.h" +#include "clang/Tooling/Inclusions/IncludeStyle.h" #include "llvm/ADT/ArrayRef.h" ---------------- Maybe `Tooling/Includes`? I think inclusions is somewhat uncommon, while includes is more widely used. Don't have a strong opinion, though. Repository: rC Clang https://reviews.llvm.org/D47068 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
