mprobst marked 2 inline comments as done. ================ Comment at: lib/Format/TokenAnnotator.cpp:698 @@ +697,3 @@ + + if (Style.Language == FormatStyle::LK_JavaScript) { + // JavaScript files can contain shebang lines of the form: ---------------- alexeagle wrote: > should we still restrict it to be on the first line? > > I suppose if you start some other line with #, that's not valid JS or TS so > it doesn't matter what clang-format does to it? Maybe it's fine like this. Done. Doesn't make a big difference I guess, but it's better to be conservative.
http://reviews.llvm.org/D20632 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits