<div>The style guide is correct. Your greps are incorrect for the correct style which is why you got &quot;0 correct uses&quot;. </div><div><br></div><div>Proof by counter example: <a href="http://trac.webkit.org/browser/trunk/WebCore/rendering/HitTestResult.cpp#L250">http://trac.webkit.org/browser/trunk/WebCore/rendering/HitTestResult.cpp#L250</a></div>
<div><br><div class="gmail_quote">On Tue, Feb 2, 2010 at 5:32 PM, Dumitru Daniliuc <span dir="ltr">&lt;<a href="mailto:dumi@chromium.org">dumi@chromium.org</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
According to <a href="http://webkit.org/coding/coding-style.html" target="_blank">http://webkit.org/coding/coding-style.html</a>, rule #5 (and the check-webkit-style script), if-statements should look like this:<br><br><span style="font-family:courier new,monospace">if (a</span><br style="font-family:courier new,monospace">

<span style="font-family:courier new,monospace">    &amp;&amp; b)</span><br style="font-family:courier new,monospace"><br>and not like this:<br><br><span style="font-family:courier new,monospace">if (a &amp;&amp;</span><br style="font-family:courier new,monospace">

<span style="font-family:courier new,monospace">    b)</span><br><br>However,<br><br><span style="font-family:courier new,monospace">find WebCore/ -name *.cpp | xargs grep -E &#39;^.*(&amp;&amp;|\|\|)\s*$&#39; | wc -l</span> = 925 incorrect uses<br>

<span style="font-family:courier new,monospace">find WebCore/ -name *.h | xargs grep -E &#39;^.*(&amp;&amp;|\|\|)\s*$&#39; | wc -l</span> = 81 incorrect uses<br>
<span style="font-family:courier new,monospace"></span><span style="font-family:courier new,monospace">find WebCore/ -name *.cpp | xargs grep -E &#39;^\s*(&amp;&amp;|\|\|).*$&#39; | wc -l</span> = 0 correct uses<br><span style="font-family:courier new,monospace">find WebCore/ -name *.h | xargs grep -E &#39;^\s*(&amp;&amp;|\|\|).*$&#39; | wc -l</span> = 0 correct uses<br>


<span style="font-family:courier new,monospace"></span><br><br>Should the rule be changed? Sorry if this was discussed already and I missed the thread.<br><br>Thanks,<br>Dumi<br><br>
<br>_______________________________________________<br>
webkit-dev mailing list<br>
<a href="mailto:webkit-dev@lists.webkit.org">webkit-dev@lists.webkit.org</a><br>
<a href="http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev" target="_blank">http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev</a><br>
<br></blockquote></div><br></div>