Wrong cookie header description in diagram.png.

RoaringBitmap / RoaringFormatSpec

Specification of the compressed-bitmap Roaring format

http://roaringbitmap.org/

Apache License 2.0

148 stars 14 forks source link

Wrong cookie header description in diagram.png. #5

Closed AAAbramenko closed 5 years ago

AAAbramenko commented 5 years ago

In diagram.png we see 4 bytes for (Cookie) and 4 bytes for (Nc = Container count). But in according to RoaringFormatSpec in case of bitmap has run containers description should be:

2 bytes for (Cookie = SERIAL_COOKIE);
2 bytes for (Nc = Container count - 1). This discrepancy makes confused.

lemire commented 5 years ago

That's correct, the diagram is misleading. I have updated it in git, but it should take some time before you can see the updated diagram due to image caching.

Re-open if an error is still present after a few hours.