xtonik

Results 9 comments of xtonik

I have written [the validation](https://github.com/RoaringBitmap/RoaringBitmap/commit/85c37485f6ace972fd0c8f99a6d03c6c90e3ef5b) according to C version of Roaring bitmaps. Some checks were removed, e.g. it is impossible to have negative capacity as it is derived from array...

I use "Intel(R) Core(TM) i5-6200U CPU @ 2.30GHz" running on Debian 11. One thing is`(char) (c - '0') < 10` vs. `'0'

Yes, you have right - within parser is the change unnoticeable. I have created separate benchmark, which gives me clear results in favour of conditional variant. But this should not...

I have found the reason - there was included "the example of how not to use `isDigit()` at all", it is rolled back by last commit caee9b2 and little difference...

It should be good to remove all dead code sections, not all were removed within #137, there are still many of them in `MappeableBitmapContainer`, it improves significantly code readability and...

The issue can be solved by simple [change](https://github.com/RoaringBitmap/RoaringBitmap/commit/1f01f5bc234d17c12f9a9635dd57cd40bbcb4616). In the change for Go version, there are three changes as the commit solves the first, the second is not relevant for...

Is it still requested? I can do that as it is simple task - mix of existing `equals()` in `(Immutable)RoaringArray` and `(Immutable)RoaringBitmap`.

Please rerun the tests, their failure was caused by network error during downloading Gradle.