djasper added inline comments.
================ Comment at: lib/Format/Format.cpp:1521 +// tokens and returns an offset after the sequence. +unsigned getOffsetAfterTokenSequence( + StringRef FileName, StringRef Code, const FormatStyle &Style, ---------------- I am somewhat hesitant to put more and more code here. Can we move some of this to a separate file? Is there a reasonable separation? ================ Comment at: lib/Format/Format.cpp:1601 + +// Returns the offset of the last #include directive after which a new +// #include can be inserted. If no such #include in the code, this returns the ---------------- I might be useful to precisely describe (here or elsewhere) what is determined here, not from an implementation standpoint, but from what a user can expect. https://reviews.llvm.org/D26909 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits