curdeius added a comment. In D99031#2697563 <https://reviews.llvm.org/D99031#2697563>, @aybassiouny wrote:
> After rechecking, turns out `AllowShortLambdasOnASingleLine` and > `BeforeLambdaBody` both need to be turned on in order for the regression to > be expressed, this affects the UT. Please update the summary with this information. ================ Comment at: clang/unittests/Format/FormatTest.cpp:2594 + // Make sure compact namespaces are not confused with lambdas + FormatStyle CompactNamespacesStyle{getLLVMStyle()}; + CompactNamespacesStyle.CompactNamespaces = true; ---------------- Nit. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D99031/new/ https://reviews.llvm.org/D99031 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits