klimek added a reviewer: hans.
klimek added a comment.

+hans

+1 to "format only current document but save all" not making much sense :)



================
Comment at: tools/clang-format-vs/ClangFormat/TypeConverterUtils.cs:7
+{
+    public sealed class TypeConverterUtils
+    {
----------------
Perhaps name this "TypeConversion" or something - I find "utils" as part of a 
name to not add anything, and it tends to accumulate cruft :)


================
Comment at: tools/clang-format-vs/ClangFormat/VsixUtils.cs:13
+{
+    internal sealed class VsixUtils
+    {
----------------
Same here, just call it Vsix? (context seems clear enough)


https://reviews.llvm.org/D29221



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

Reply via email to