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

New function proposal: "roaring_move_uint32_iterator_equalorsmaller" #344

Open
dtenny opened this issue Jan 17, 2022 · 2 comments
Open

New function proposal: "roaring_move_uint32_iterator_equalorsmaller" #344

dtenny opened this issue Jan 17, 2022 · 2 comments

Comments

@dtenny
Copy link

dtenny commented Jan 17, 2022

Basically like roaring_move_uint32_iterator_equalorlarger, except that its purpose is to move through elements in descending order instead of ascending order.

For use in conjunction with roaring_init_iterator_last.

Seem like an oversight that the proposed function is missing. For now my foreign function wrappers call roaring_init_iterator_last and then roaring_previous_uint32_iterator until the desired values are found, but this is not efficient.

@lemire
Copy link
Member

lemire commented Jan 18, 2022

@dtenny Would you be willing to produce a pull request ?

@dtenny
Copy link
Author

dtenny commented Jan 18, 2022

Hi, I know, that'd be the civilized thing to do :-) I'm pretty focused on the lisp side of things at the moment, so it's unlikely I'll find time to start with C PR's. I'll let you know if I decide to make a run at it though. If you're curious what the lisp apis look like you can find it in my cl-roaring repo. Still a WIP though, enough to be useful to lispers wanting to use CRoaring, but not necessarily complete.

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

No branches or pull requests

2 participants