Skip to content

feat(retain): add retain method - #366

Open
HellButcher wants to merge 1 commit into
RoaringBitmap:mainfrom
HellButcher:feat/retain
Open

HellButcher wants to merge 1 commit into
RoaringBitmap:mainfrom
HellButcher:feat/retain

Conversation

@HellButcher

Copy link
Copy Markdown

adds a retain(&mut self, f: FnMut(u32) -> bool) method to RoaringBitmap and RoaringTreemap.
It works similar to Vec::retain.

@HellButcher
HellButcher force-pushed the feat/retain branch 2 times, most recently from 31a2549 to cc2f119 Compare September 15, 2026 10:41

@Kerollmops Kerollmops left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey @HellButcher 👋

Thank you for the nice addition. Would you mind fixing the CI errors, please?

@Kerollmops Kerollmops left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Final little clippy issue now 👀 😊

adds a `retain(&mut self, f: FnMut(u32) -> bool)` method to
`RoaringBitmap` and `RoaringTreemap`.
It works similar to `Vec::retain`.

This branch has not been deployed

No deployments
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