Skip to content
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.

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

Add roaring_bitmap_deserialize_safe #486

Merged
merged 3 commits into from
Jun 5, 2023
Merged

Conversation

SalvatorePreviti
Copy link
Contributor

@SalvatorePreviti SalvatorePreviti commented Jun 2, 2023

CRoaring offers roaring_bitmap_portable_deserialize_safe but it does not provide roaring_bitmap_deserialize_safe.

  • Add new function roaring_bitmap_t *roaring_bitmap_deserialize_safe(const void *buf, size_t maxbytes);
  • Add some basic unit tests for it

@lemire
Copy link
Member

lemire commented Jun 2, 2023

That's reasonable.

@SalvatorePreviti SalvatorePreviti requested a review from a team June 2, 2023 22:28
@lemire
Copy link
Member

lemire commented Jun 5, 2023

Will be part of the next release.

@lemire lemire merged commit e8c0bca into master Jun 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants