<div class="gmail_quote">On Fri, May 6, 2011 at 1:38 PM, Richard Trieu <span dir="ltr"><<a href="mailto:rtrieu@google.com">rtrieu@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>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>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> </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" class="cremed">http://codereview.appspot.com/4444068/</a></div></blockquote></div><br><div>
Some minor comments left on the patch here.</div>