Oops, here's the correct patch.<div><br><div class="gmail_quote">On Sun, Feb 5, 2012 at 4:49 PM, Talin <span dir="ltr"><<a href="mailto:viridia@gmail.com">viridia@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

Done.<div><div></div><div class="h5"><br><br><div class="gmail_quote">On Sun, Feb 5, 2012 at 4:43 PM, Jakob Stoklund Olesen <span dir="ltr"><<a href="mailto:stoklund@2pi.dk" target="_blank">stoklund@2pi.dk</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div><br>
On Feb 5, 2012, at 4:31 PM, Talin wrote:<br>
<br>
> Patch to refactor the hashing functions for the constant unique map, to avoid code duplication.<br>
<br>
</div>Wouldn't it give nicer code if you kept the intermediate state as a class member?<br>
<br>
Something like:<br>
<br>
IncrementalHash Hash<br>
Hash.add(foo);<br>
Hahs.add(bar);<br>
return Hash.finish();<br>
<font color="#888888"><br>
/jakob<br>
<br>
</font></blockquote></div><br><br clear="all"><div><br></div></div></div><font color="#888888">-- <br>-- Talin<br>
</font></blockquote></div><br><br clear="all"><div><br></div>-- <br>-- Talin<br>
</div>