[osmosis-dev] bug in read simple task, integer overflow

[Brett Henderson]

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.

I'm not sure why these were missed, but hopefully this will be the last of
the 64-bit issues.


On 5 May 2013 20:46, Martin Schafran <martin at ampelmeter.com> wrote:

> the BitSetIdTracker must be enhanced anyway to support 64 bit.
>
> i guess we have to split the /BitSet positiveSet/ into lower and upper
> sets,
> right? And remove the LongAsInt class and other long to int and vice versa
> conversions.
>
> martin
>
>
>
>
> _______________________________________________
> osmosis-dev mailing list
> osmosis-dev at openstreetmap.org
> http://lists.openstreetmap.org/listinfo/osmosis-dev
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/osmosis-dev/attachments/20130720/ecf2023c/attachment.html>