<div dir="ltr">On 31 July 2013 18:07, Ian Romanick <span dir="ltr"><<a href="mailto:idr@freedesktop.org" target="_blank">idr@freedesktop.org</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

Patches 7, 9 (with Ken's suggestions), 10, 12, 13, 14, 15 (with the one nit accounted... 18 and perhaps other have the same issue), 17, 18, 19, 20, 21, 22, 27, 28, 29 (though my question may affect 29 & 30), 30, 31, 32 (with minor nits), 33, and 34 (with a minor nit).<br>


<br>
Reviewed-by: Ian Romanick <<a href="mailto:ian.d.romanick@intel.com" target="_blank">ian.d.romanick@intel.com</a>><br></blockquote><div><br></div><div>Wow, thanks for all the review!  I'll go through your comments in detail tomorrow.<br>

</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
I don't feel like I understand 16 very well.  Perhaps we can talk about it tomorrow?</blockquote><div><br></div><div>Sure, no problem.  To my knowledge, no one else but me has ever touched the varying packing code, so it's possible that there are some Paulisms in there that make the code hard to understand for anyone but me :)<br>
</div></div></div></div>