curdeius added inline comments.
================
Comment at: tools/clang-format-vs/ClangFormat/ClangFormatPackage.cs:57
@@ +56,3 @@
+ {
+ StandardValuesCollection svc = new
StandardValuesCollection(values);
+ return svc;
----------------
aaron.ballman wrote:
> Can we just return directly here instead of storing in a local?
Hmm, yes, we can.
================
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" +
----------------
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.
================
Comment at: tools/clang-format-vs/ClangFormat/ClangFormatPackage.cs:116
@@ +115,3 @@
+ // Check if string contains quotes.
+ if (s.IndexOf('\"') != -1)
+ throw new NotSupportedException("Filename cannot contain
quotes");
----------------
aaron.ballman wrote:
> I see now why this is being used, but it took a lot of context from review
> comments to understand. Can you please put in a more descriptive comment
> about why we're checking for quotes, despite them being invalid filename
> characters on Windows?
OK, will do.
http://reviews.llvm.org/D13549
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
