<div dir="ltr"><div class="gmail_extra">I have no objection to us using this compiler feature.<br><br><div class="gmail_quote">On Fri, Feb 22, 2013 at 2:35 PM, Julien Chaffraix <span dir="ltr">&lt;<a href="mailto:jchaffraix@webkit.org" target="_blank">jchaffraix@webkit.org</a>&gt;</span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi WebKit folks,<br>
<br>
over several reviews, I have been saying that the following line is a<br>
coding style violation:<br>
<br>
firstVariable = secondVariable = 0;<br>
<br>
For a concrete example, the computePreferredLogicalWidths uses the<br>
following pattern:<br>
<br>
minWidth = maxWidth = max&lt;int&gt;(minWidth, tableLogicalWidth.value());<br>
<br>
My justification is that those are 2 statements and thus should be on<br>
2 lines per &quot;Each statement should get its own line.&quot;. Some people<br>
consider that the previous rule doesn&#39;t apply to multiple assignments<br>
on one line and that such code is fine by the book.<br>
<br>
What do people think?<br>
<br>
Thanks,<br>
Julien<br>
_______________________________________________<br>
webkit-dev mailing list<br>
<a href="mailto:webkit-dev@lists.webkit.org">webkit-dev@lists.webkit.org</a><br>
<a href="https://lists.webkit.org/mailman/listinfo/webkit-dev" target="_blank">https://lists.webkit.org/mailman/listinfo/webkit-dev</a><br>
</blockquote></div><br></div></div>