New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix serialization when serializing into a vec with contents #111

Merged

saulius merged 1 commit into RoaringBitmap:master from Dr-Emann:serialize_into_existing_data

Jul 17, 2023

Member

Dr-Emann commented Jul 4, 2023

We weren't taking into account existing data in serialize_into implementations. This fixes that, and adds tests for this case.


          Fix serialization when serializing into a vec with contents

c7ca8bc

Member

saulius commented Jul 11, 2023 •

edited

Loading

Whoops, good catch!

saulius self-requested a review

July 11, 2023 14:06

saulius approved these changes

View reviewed changes

saulius merged commit f7882e6 into RoaringBitmap:master

Dr-Emann deleted the serialize_into_existing_data branch

August 5, 2023 08:46

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet