<div>Patch to completely remove warning for mixed signedness of conditional operands.</div><div><br></div>Still available at <a href="http://codereview.appspot.com/4731042/">http://codereview.appspot.com/4731042/</a><br><br>
<div class="gmail_quote">On Thu, Jul 14, 2011 at 4:30 PM, 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="im"><div class="gmail_quote">On Thu, Jul 14, 2011 at 4:24 PM, Richard Trieu <span dir="ltr"><<a href="mailto:rtrieu@google.com" target="_blank">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">

The main motivation is to make -Wsign-compare only about checking signs during comparisons.  Since the operands of the conditional operator aren't compared, it seems out of place to check it there.</blockquote></div>
<br>
</div><div>So, howabout a patch to just remove this part of -Wsign-compare? I mean, if there is a good reason to keep it, then cool, I just don't think we need another warning here...</div>
</blockquote></div><br>