<div class="gmail_quote">On Mon, Feb 27, 2012 at 2:59 AM, James Molloy <span dir="ltr"><<a href="mailto:James.Molloy@arm.com">James.Molloy@arm.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Please? Anyone? Ping? :(<br></blockquote><div><br></div><div>Sorry... I'm trying to emerge from my February rockpile... And it happens I can finish this one easily now that I have a moment...</div><div><br></div><div>
The patch looks great semantically. A couple of nit-picks. Just commit once these are fixed. =]</div><div><br></div><div>1) Don't set all the properties on the new test files.</div><div><br></div><div>2) Use the more conventional capitalized initialism loop variable naming convention: I, E, DI, DE, etc.</div>
<div><br></div><div><br></div><div>Sorry for the mess of review, I shouldn't vanish for 4 weeks at a time...</div></div>