Cleanup roaring integration code

Mainly fix code style, rename variables, clean up some logic
I don't see 0.4.3 in maven central yet, guess I'll have to wait a bit for
it to show up.

On Fri, Oct 24, 2014 at 5:21 PM, Xavier Léauté <xavier@metamarkets.com>
wrote:

> Awesome, thanks for the quick turnaround!
>
> On Fri, Oct 24, 2014 at 5:08 PM, Daniel Lemire <notifications@github.com>
> wrote:
>
>> In src/test/java/com/metamx/collections/bitmap/BitmapBenchmarkTest.java:
>>
>> > +      ConciseSet c = new ConciseSet();
>> > +      MutableRoaringBitmap r = new MutableRoaringBitmap();
>> > +      for(int k = 0; k < LENGTH; ++k) {
>> > +        if(rand.nextDouble() < .1) {
>> > +          c.add(k);
>> > +          r.add(k);
>> > +          expectedUnion.set(k);
>> > +        }
>> > +      }
>> > +      for(int k = 0; k < knownTrue.length; ++k) {
>> > +        c.add(knownTrue[k]);
>> > +        r.add(knownTrue[k]);
>> > +        expectedUnion.set(knownTrue[k]);
>> > +      }
>> > +      concise[i] = ImmutableConciseSet.newImmutableFromMutable(c);
>> > +      int size = r.serializedSizeInBytes();
>>
>> @xvrl <https://github.com/xvrl> I have tracked down to issue to a rarely
>> occurring serialization problem. This is fixed in version 0.4.3 of Roaring
>> released just now. After you upgrade, this issue should go away.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/metamx/bytebuffer-collections/pull/4/files#r19371237>
>> .
>>
>
>