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

Reply via email to