<div dir="auto"><div>The old loop seemed confusing to me and it overestimated the size of the memory to allocate by a large factor. The old loop would need to have about half of it rewritten to fix the size overestimation and a little more for fixing the original bug. If you think it's very important to keep as much of the old code as possible, I can rewrite the patch that way, but I don't think that's necessary.</div><div dir="auto">Jacob Lifshay</div><div dir="auto"><div class="gmail_extra" dir="auto"><br><div class="gmail_quote">On Feb 7, 2017 7:43 PM, "Vincent Povirk" <<a href="mailto:madewokherd@gmail.com">madewokherd@gmail.com</a>> wrote:<br type="attribution"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Is it really necessary to rewrite this loop? It'd be better to fix the<br>
existing code, if possible.<br>
</blockquote></div><br></div></div></div>