<div dir="ltr">Bump.  This looks pretty benign to me so I'll probably go ahead and commit without review if I don't hear back in the next, say, 6 hours or so.</div><div class="gmail_extra"><br><br><div class="gmail_quote">
On Thu, Jul 17, 2014 at 9:47 AM, Zachary Turner <span dir="ltr"><<a href="mailto:zturner@google.com" target="_blank">zturner@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 dir="ltr">I accidentally reversed the conditional on this patch.  A corrected patch is attached.<div><br></div><div><br></div></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><br><div class="gmail_quote">
On Wed, Jul 16, 2014 at 1:42 PM, Zachary Turner <span dir="ltr"><<a href="mailto:zturner@google.com" target="_blank">zturner@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 dir="ltr">Hi Greg,<div><br></div><div>Looks like a simple bug where it should have been "a | b | (c ? 0 : d)" instead of "a | b | c ? 0 : d".  Rather than rely on order of operations, which I always find confusing, I opted to just move this out into explicit initialization of the flags.</div>


<div><br></div><div>Let me know if the new code matches your original intent.</div></div>
</blockquote></div><br></div>
</div></div></blockquote></div><br></div>