I think /* explanation of what I&#39;m doing */ is strictly worse than readableCode(UnderstandableParameter).<div><br></div><div>I&#39;d rather have readable code than comments attempting to excuse unreadable code.</div><div>
<br></div><div>-eric</div><div><br><div class="gmail_quote">On Fri, Dec 3, 2010 at 1:57 PM, Antonio Gomes <span dir="ltr">&lt;<a href="mailto:tonikitoo@gmail.com">tonikitoo@gmail.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
I do not think that XXX:repaint(true /* immediate */) is so bad<br>
either, if I understood Hyatt&#39;s comment correctly, and I agree with him on it.<br>
<br>
Having a enum is ideal, but no need for it to be mandatory, as other<br>
also pointed out.<br>
<div><div></div><div class="h5"><br>
On Fri, Dec 3, 2010 at 4:54 PM, Eric Seidel &lt;<a href="mailto:eric@webkit.org">eric@webkit.org</a>&gt; wrote:<br>
&gt; I&#39;m not sure we have any examples of bool passing like that in real code.<br>
&gt; The case I&#39;m concerned about is not one of single argument bools:<br>
&gt; doSoemthing(bool)<br>
&gt; but more of multi-argument functions:<br>
&gt; doSomething(something, bool)<br>
&gt; I&#39;m trying to write a rule which can be easily automated by<br>
&gt; check-webkit-style.<br>
&gt; It&#39;s possible we could tighten the rule further to only allow<br>
&gt; single-argument bools where &quot;set&quot; is in the function name.<br>
&gt; It sounds like most folks are in agreement.  We should add a rule like this<br>
&gt; to check-webkit-style.  Sounds like Dave Levin may already have something<br>
&gt; partial in the works.<br>
&gt; -eric<br>
&gt; On Fri, Dec 3, 2010 at 1:45 PM, Ryosuke Niwa &lt;<a href="mailto:rniwa@webkit.org">rniwa@webkit.org</a>&gt; wrote:<br>
&gt;&gt;<br>
&gt;&gt; On Fri, Dec 3, 2010 at 1:37 PM, David Hyatt &lt;<a href="mailto:hyatt@apple.com">hyatt@apple.com</a>&gt; wrote:<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; The only exception I would make to this rule is if all the call sites use<br>
&gt;&gt;&gt; variables and never pass in raw true or false.  In that case there&#39;s no loss<br>
&gt;&gt;&gt; of readability, and whether you use an enum vs. a bool is irrelevant.<br>
&gt;&gt;&gt; I think in general the rule should be &quot;Keep your call sites readable, and<br>
&gt;&gt;&gt; convert to enums if you find that the call sites are becoming inscrutable.&quot;<br>
&gt;&gt;<br>
&gt;&gt; That rule makes sense to me.<br>
&gt;&gt; On Fri, Dec 3, 2010 at 1:40 PM, Eric Seidel &lt;<a href="mailto:eric@webkit.org">eric@webkit.org</a>&gt; wrote:<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; Dave, I&#39;m not sure I understand your exception.  Could you give an<br>
&gt;&gt;&gt; example?<br>
&gt;&gt;<br>
&gt;&gt; I think what he means is that<br>
&gt;&gt; bool doSomething();<br>
&gt;&gt; void doSomethingElse(bool);<br>
&gt;&gt; and the only case we always call doSomethingElse with a return value of<br>
&gt;&gt; some function or with a variable:<br>
&gt;&gt; doSomethingElse(doSomething());<br>
&gt;&gt; doSomethingElse(shouldNotDoSomethingElse);<br>
&gt;&gt; etc...<br>
&gt;&gt; and we never call it with raw true/false:<br>
&gt;&gt; doSomethingElse(true)<br>
&gt;&gt; doSomethingElse(false)<br>
&gt;&gt; - Ryosuke<br>
&gt;&gt;<br>
&gt;&gt; _______________________________________________<br>
&gt;&gt; webkit-dev mailing list<br>
&gt;&gt; <a href="mailto:webkit-dev@lists.webkit.org">webkit-dev@lists.webkit.org</a><br>
&gt;&gt; <a href="http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev" target="_blank">http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev</a><br>
&gt;&gt;<br>
&gt;<br>
&gt;<br>
&gt; _______________________________________________<br>
&gt; webkit-dev mailing list<br>
&gt; <a href="mailto:webkit-dev@lists.webkit.org">webkit-dev@lists.webkit.org</a><br>
&gt; <a href="http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev" target="_blank">http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev</a><br>
&gt;<br>
&gt;<br>
<br>
<br>
<br>
</div></div>--<br>
<font color="#888888">--Antonio Gomes<br>
</font></blockquote></div><br></div>