HazardyKnusperkeks added inline comments.
================ Comment at: clang/lib/Format/Format.cpp:1304 LLVMStyle.RequiresClausePosition = FormatStyle::RCPS_OwnLine; + LLVMStyle.RequiresExpressionIndentation = FormatStyle::REI_Keyword; LLVMStyle.SeparateDefinitionBlocks = FormatStyle::SDS_Leave; ---------------- owenpan wrote: > HazardyKnusperkeks wrote: > > owenpan wrote: > > > Delete it. > > Why delete? > > Set it to OuterScope. > > Why delete? > > Set it to OuterScope. > > Cuz OuterScope is the default. :) The default is uninitialized. Not setting anything here is undefined behavior. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D129443/new/ https://reviews.llvm.org/D129443 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
