Wawha added a comment. In D104222#2819358 <https://reviews.llvm.org/D104222#2819358>, @MyDeveloperDay wrote:
> @Wawha your cases could be covered by the following (in mustBreakBefore) > > if (Style.BraceWrapping.BeforeLambdaBody && Right.is(TT_LambdaLBrace) && > Left.isOneOf(tok::star, tok::amp, tok::ampamp)) { > return true; > } > > As I think its the presence of & and * that causes it to not wrap. > > By all means if you would prefer to commandeer the patch I'm ok with that. I > was just on a bug fixing day. If you'd like to take this over and add the > unit tests @christophe-calmejane suggest, I'm happy to be the reviewer Indeed, & and * could be the root cause. About fixing that, I won't have the time in the next days. It will take time for me to re-read and re-understand that code. But I could find time to test patch or modification in my code to check that there is no regression. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D104222/new/ https://reviews.llvm.org/D104222 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits