On Fri, Sep 7, 2012 at 12:07 PM, Adrian Buehlmann <span dir="ltr"><<a href="mailto:adrian@cadifra.com" target="_blank">adrian@cadifra.com</a>></span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">Bryan, can I ask you what "slop" means?</div></blockquote><div><br></div><div>In this context, it means "leftovers". It serves the same purpose as this code in store.py: <a href="http://selenic.com/repo/hg/file/3ee5d3c372fa/mercurial/store.py#l218">http://selenic.com/repo/hg/file/3ee5d3c372fa/mercurial/store.py#l218</a></div>
<div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="h5">If I would have had to write such kind of code, I would be almost 100%</div></div>
sure *I* would have (unintentionally) inserted at least 5 bugs in it.<br></blockquote><div><br></div><div>Me too, since I fixed about that many while writing it, mainly due to all the extra test cases you added to test-hybridencode.py.</div>
<div><br></div><div>I'm not certain there are no remaining bugs, but the current code is quite a lot closer to correct.</div><div><br></div><div>I'll be writing a fuzz tester to exercise this code further.</div></div>