[PATCH] D73335: clang-format: [JS] options for arrow functions.

Krasimir Georgiev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jan 24 05:37:46 PST 2020


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





More information about the cfe-commits mailing list