krasimir added inline comments.
================ Comment at: clang/lib/Format/UnwrappedLineParser.cpp:906 // trailing semicolons from other constructs split to several lines. -static bool tokenCanStartNewLine(const clang::Token &Tok) { +static bool tokenCanStartNewLine(const FormatToken *Tok) { // Semicolon can be a null-statement, l_square can be a start of a macro or ---------------- nit: pass `Tok` by const reference (makes sure we don't have to deal with nullptr and as bonus will make this patch diff smaller). CHANGES SINCE LAST ACTION https://reviews.llvm.org/D79990/new/ https://reviews.llvm.org/D79990 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits