krasimir added a comment. > FYI I've started a conversation on whether SLS_All should be the default > Google style. But the option I think is useful in either case.
If that's the case, consider adding the option to the style in a follow-up patch: GoogleStyle.AllowShortLambdasOnASingleLine = FormatStyle::SLS_All; that would have an advantage of making clear which test case diffs are result of the implementation and which are result of the style flip. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73335/new/ https://reviews.llvm.org/D73335 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits