<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Aug 26, 2015 at 7:03 PM, Florian Lindner <span dir="ltr"><<a href="mailto:mailinglists@xgm.de" target="_blank" class="cremed">mailinglists@xgm.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><span><br>
Am Mittwoch, 26. August 2015, 18:40:43 schrieb Daniel Jasper:<br>
> Sure, there is <a href="http://bugs.llvm.org" rel="noreferrer" target="_blank" class="cremed">bugs.llvm.org</a> with a dedicated "Formatter" component.<br>
><br>
> However, the bar for adding such features is:<br>
> - Large project with open style guide recommending it.<br>
<br>
</span>Why that? Of course, I'm not suggesting to change the default setting, but I fail to understand how adding such a feature hurts if it doesn't change the default.<br></blockquote><div><br></div><div>Every configuration flag has a certain cost. Among other things, this includes:</div><div><div>- Style flags become less discoverable</div><div>- Maintenance increases when ensuring that all combinations of flags work as expected</div></div><div><br></div><div>I'll try to put a more detailed explanation of the reasons on some documentation page.</div><div><br></div><div>In essence, we'd rather support a stricter set of rules really well instead trying to make clang-format work for everyone but doing a bad job of it.</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><span>
> - Person willing to contribute an maintain a patch.<br>
<br>
</span>Sure.<br>
<div><div><br>
Best Regards,<br>
Florian<br>
<br>
><br>
><br>
> On Wed, Aug 26, 2015 at 5:40 PM, Florian Lindner <<a href="mailto:mailinglists@xgm.de" target="_blank" class="cremed">mailinglists@xgm.de</a>><br>
> wrote:<br>
><br>
> > Am Mittwoch, 26. August 2015, 13:11:32 schrieb Daniel Jasper:<br>
> > > No, there isn't.<br>
> ><br>
> > Ok, thanks for that statement. Is this functionality planned somehow? Is<br>
> > there a bug tracker where I can file this as a wishlist item?<br>
> ><br>
> > Best Regards,<br>
> > Florian<br>
> ><br>
> > ><br>
> > > On Wed, Aug 26, 2015 at 11:23 AM, Florian Lindner via cfe-dev <<br>
> > > <a href="mailto:cfe-dev@lists.llvm.org" target="_blank" class="cremed">cfe-dev@lists.llvm.org</a>> wrote:<br>
> > ><br>
> > > > Hello,<br>
> > > ><br>
> > > > I have code like that:<br>
> > > ><br>
> > > > ierr = KSPDestroy(&ksp);CHKERRQ(ierr);<br>
> > > ><br>
> > > > clang-format introduces a linebreak between the two statements. Is<br>
> > there a<br>
> > > > possiblity to omit that?<br>
> > > ><br>
> > > > My configuration, based on style LLVM:<br>
> > > ><br>
> > > > AllowAllParametersOfDeclarationOnNextLine: true<br>
> > > > AllowShortBlocksOnASingleLine: true<br>
> > > > AllowShortCaseLabelsOnASingleLine: true<br>
> > > > AllowShortIfStatementsOnASingleLine: true<br>
> > > > AllowShortLoopsOnASingleLine: true<br>
> > > > AllowShortFunctionsOnASingleLine: All<br>
> > > > AlwaysBreakAfterDefinitionReturnType: false<br>
> > > > AlwaysBreakTemplateDeclarations: false<br>
> > > > AlwaysBreakBeforeMultilineStrings: false<br>
> > > ><br>
> > > > (I've just set every remotely related option to don't break)<br>
> > > ><br>
> > > > Thanks,<br>
> > > > Florian<br>
> > > > _______________________________________________<br>
> > > > cfe-dev mailing list<br>
> > > > <a href="mailto:cfe-dev@lists.llvm.org" target="_blank" class="cremed">cfe-dev@lists.llvm.org</a><br>
> > > > <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev" rel="noreferrer" target="_blank" class="cremed">http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev</a><br>
> > > ><br>
> ><br>
</div></div></blockquote></div><br></div></div>