Cleanup roaring integration code

Mainly fix code style, rename variables, clean up some logic
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>.
>