curdeius added inline comments.
================ Comment at: clang/lib/Format/UnwrappedLineParser.cpp:874 - if (SimpleBlock && !KeepBraces) { + auto RemoveBraces = [=]() mutable { + if (KeepBraces || !SimpleBlock) ---------------- Are there many captures here? Wouldn't it be better to be explicit and/or capture by ref? Do we need a mutable lambda? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D126358/new/ https://reviews.llvm.org/D126358 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits