owenpan accepted this revision. owenpan added a comment. Thanks! LGTM other than the extraneous braces on lines 1859-1861. (See https://llvm.org/docs/CodingStandards.html#don-t-use-braces-on-simple-single-statement-bodies-of-if-else-loop-statements.)
================ Comment at: clang/lib/Format/UnwrappedLineParser.cpp:1859 + nextToken(); + if (FormatTok->isNot(tok::l_brace)) { + return false; ---------------- Remove braces. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D115738/new/ https://reviews.llvm.org/D115738 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits