On Mon, Dec 06, 2010 at 09:24:24PM -0800, Justin Pettit wrote: > Looks fine. I think it would be nice to add to the comment that > parsing the field name as a 32-bit hex integer is done for tests; > otherwise it looks really odd. OK, I made that change and will push this out soon. Thanks for the reviews.