[PATCH] D13549: Added new options to ClangFormat VSIX package.
Aaron Ballman via cfe-commits
cfe-commits at lists.llvm.org
Tue Oct 13 07:40:38 PDT 2015
aaron.ballman added inline comments.
================
Comment at: tools/clang-format-vs/ClangFormat/ClangFormatPackage.cs:82
@@ -42,1 +81,3 @@
+ " - Predefined styles ('LLVM', 'Google', 'Chromium', 'Mozilla', 'WebKit').\n" +
+ " - 'file' to search for a YAML .clang-format or _clang-format\n" +
" configuration file.\n" +
----------------
curdeius wrote:
> aaron.ballman wrote:
> > Why is 'file' now lowercased?
> I wanted to be consistent with clang-format --help. Besides, lowercase styles (file, none) are somewhat special and it's nice to have them stand out, IMO.
That seems reasonable to me. Just making sure it wasn't a drive-by unintended change, or had some weird semantic meaning in MSVC.
http://reviews.llvm.org/D13549
More information about the cfe-commits
mailing list