<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Sep 17, 2018 at 10:00 AM, Ravishankar N <span dir="ltr">&lt;<a href="mailto:ravishankar@redhat.com" target="_blank">ravishankar@redhat.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class=""><br>
On 09/13/2018 03:34 PM, Niels de Vos wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On Thu, Sep 13, 2018 at 02:25:22PM +0530, Ravishankar N wrote:<br>
...<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
What rules does clang impose on function/argument wrapping and alignment? I<br>
somehow found the new code wrapping to be random and highly unreadable. An<br>
example of &#39;before and after&#39; the clang format patches went in:<br>
<a href="https://paste.fedoraproject.org/paste/dC~aRCzYgliqucGYIzxPrQ" rel="noreferrer" target="_blank">https://paste.fedoraproject.or<wbr>g/paste/dC~aRCzYgliqucGYIzxPrQ</a> Wondering if<br>
this is just me or is it some problem of spurious clang fixes.<br>
</blockquote>
I agree that this example looks pretty ugly. Looking at random changes<br>
to the code where I am most active does not show this awkward<br>
formatting.<br>
</blockquote>
<br></span>
So one of my recent patches is failing smoke and clang-format is insisting [<a href="https://build.gluster.org/job/clang-format/22/console" rel="noreferrer" target="_blank">https://build.gluster.org/job<wbr>/clang-format/22/console</a>] on wrapping function arguments in an unsightly manner. Should I resend my patch with this new style of wrapping ?<br>
<br></blockquote><div><br></div><div>I would say yes! We will get better, by changing options of clang-format once we get better options there. But for now, just following the option suggested by clang-format job is good IMO.</div><div><br></div><div>-Amar</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Regards,<br>
Ravi<div class="HOEnZb"><div class="h5"><br>
<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
However, I was expecting to see enforcing of the<br>
single-line-if-statements like this (and while/for/.. loops):<br>
<br>
     if (need_to_do_it) {<br>
          do_it();<br>
     }<br>
<br>
instead of<br>
<br>
     if (need_to_do_it)<br>
          do_it();<br>
<br>
At least the conversion did not take care of this. But, maybe I&#39;m wrong<br>
as I can not find the discussion in <a href="https://bugzilla.redhat.com/1564149" rel="noreferrer" target="_blank">https://bugzilla.redhat.com/15<wbr>64149</a><br>
about this. Does someone remember what was decided in the end?<br>
<br>
Thanks,<br>
Niels<br>
</blockquote>
<br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div>Amar Tumballi (amarts)<br></div></div></div></div></div>
</div></div>