<div dir="ltr"><div>Three points in the codebase were still using the BitSetIdTracker.  I've just replaced these with DynamicIdTracker which is 64-bit capable.  BitSetIdTracker will never be 64-bit capable, at least not in its current form.  Note that DynamicIdTracker uses BitSetIdTracker as part of its internal implementation, but ensures that no single instance exceeds a 32-bit range of ids.<br>
<br></div><div>I'm not sure why these were missed, but hopefully this will be the last of the 64-bit issues.<br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On 5 May 2013 20:46, Martin Schafran <span dir="ltr"><<a href="mailto:martin@ampelmeter.com" target="_blank">martin@ampelmeter.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">the BitSetIdTracker must be enhanced anyway to support 64 bit.<br>
<br>
i guess we have to split the /BitSet positiveSet/ into lower and upper sets,<br>
right? And remove the LongAsInt class and other long to int and vice versa<br>
conversions.<br>
<span class="HOEnZb"><font color="#888888"><br>
martin<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
<br>
<br>
<br>
_______________________________________________<br>
osmosis-dev mailing list<br>
<a href="mailto:osmosis-dev@openstreetmap.org">osmosis-dev@openstreetmap.org</a><br>
<a href="http://lists.openstreetmap.org/listinfo/osmosis-dev" target="_blank">http://lists.openstreetmap.org/listinfo/osmosis-dev</a><br>
</div></div></blockquote></div><br></div>