<div class="gmail_quote">On Fri, Dec 3, 2010 at 1:37 PM, David Hyatt <span dir="ltr">&lt;<a href="mailto:hyatt@apple.com">hyatt@apple.com</a>&gt;</span> wrote:<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">

<div style="word-wrap:break-word"><div>The only exception I would make to this rule is if all the call sites use variables and never pass in raw true or false.  In that case there&#39;s no loss of readability, and whether you use an enum vs. a bool is irrelevant.</div>

<div><br></div><div>I think in general the rule should be &quot;Keep your call sites readable, and convert to enums if you find that the call sites are becoming inscrutable.&quot;</div></div></blockquote><div><br></div><div>

That rule makes sense to me.</div><div><br></div><div>On Fri, Dec 3, 2010 at 1:40 PM, Eric Seidel <span dir="ltr">&lt;<a href="mailto:eric@webkit.org">eric@webkit.org</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0.8ex; border-left-width: 1px; border-left-color: rgb(204, 204, 204); border-left-style: solid; padding-left: 1ex; ">

Dave, I&#39;m not sure I understand your exception.  Could you give an example?</blockquote><div><br></div><div>I think what he means is that</div><div>bool doSomething();</div><div>void doSomethingElse(bool);</div><div>
<br>
</div><div>and the only case we always call doSomethingElse with a return value of some function or with a variable:</div><div>doSomethingElse(doSomething());</div><div>doSomethingElse(shouldNotDoSomethingElse);</div><div>

etc...</div><div><br></div><div>and we never call it with raw true/false:</div><div>doSomethingElse(true)</div><div>doSomethingElse(false)</div><div><br></div></div><div>- Ryosuke</div><div><br></div></div>