<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Jul 28, 2014 at 12:46 PM, Quentin Colombet <span dir="ltr"><<a href="mailto:qcolombet@apple.com" target="_blank">qcolombet@apple.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div id=":4nm" class="a3s" style="overflow:hidden">I think we still have a bug in the current implementation.<br>
See my inline comments.<br></div></blockquote><div><br></div><div>Seems likely. Can you craft a test case to demonstrate it?</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div id=":4nm" class="a3s" style="overflow:hidden">
<br>
Also, I think it would be nice to update the comments on the masks value instead of removing them.<br>
When I reversed engineered this code, it took me ages to figure out what were these magic values.</div></blockquote></div><br>I'm not sure that's going to help. The problem with comments is that they can get out of sync. I also don't think that little endian offsets are surprising or need lots of clarification here? The 1 instead of 0 thing you point out screams wrong to me at a glance, and I don't think a comment would really make it better (or worse, unless it got out of date).</div>
</div>