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