rdwampler marked an inline comment as done. rdwampler added inline comments.
================ Comment at: clang/lib/Format/Format.cpp:649 LLVMStyle.AllowShortIfStatementsOnASingleLine = false; + LLVMStyle.AllowShortLambdasOnASingleLine = FormatStyle::SLS_All; LLVMStyle.AllowShortLoopsOnASingleLine = false; ---------------- MyDeveloperDay wrote: > What is the difference between what clang-format does now and using SLS_All, > i.e. if your introducing a new style shouldn't the default be to not change > the exsiting code? > > Without trying this myself I would think this needs to be SLS_None? (am I > wrong?) AFAICT, the current behavior is to always put lambdas on a single line, if possible. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57687/new/ https://reviews.llvm.org/D57687 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
