<div class="gmail_quote">On Thu, May 26, 2011 at 3:18 AM, Chandler Carruth <span dir="ltr"><<a href="mailto:chandlerc@google.com">chandlerc@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div class="gmail_quote"><div class="im">On Fri, May 6, 2011 at 1:38 PM, Richard Trieu <span dir="ltr"><<a href="mailto:rtrieu@google.com" target="_blank">rtrieu@google.com</a>></span> wrote:<br></div><div class="im">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div>The old warning that checked for logical operators with constants did not treat 1 and 0 as it would for other constants.  Changed the warning so that if a bool type exists, 1 and 0 will trigger the warning.</div></blockquote>

<div><br></div></div><div>Is there a PR for this? It'd be good to have more motivating examples in the description or something... But you do have one in the Code Review description I see.</div><div class="im"><div> </div>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div>Patch is attached and available through Code Review:</div><div><a href="http://codereview.appspot.com/4444068/" target="_blank">http://codereview.appspot.com/4444068/</a></div></blockquote></div></div><br><div>
Some minor comments left on the patch here.</div>
</blockquote></div><meta http-equiv="content-type" content="text/html; charset=utf-8">Made the changes that Chandler suggested in Code Review.  Anyone else have any comments on this patch?