[PATCH] D57687: [clang-format] Add style option AllowShortLambdasOnASingleLine
Ronald Wampler via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Mar 20 21:00:52 PDT 2019
rdwampler added inline comments.
================
Comment at: clang/lib/Format/TokenAnnotator.cpp:2976
+
+ return Style.AllowShortLambdasOnASingleLine == FormatStyle::SLS_None ||
+ Style.AllowShortLambdasOnASingleLine == FormatStyle::SLS_Inline ||
----------------
klimek wrote:
> If SLS_All is supposed to not change anything, should we fall through here if (SLS_All)?
Yes, I think that's a better approach to ensure the current behavior is unchanged.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D57687/new/
https://reviews.llvm.org/D57687
More information about the cfe-commits
mailing list