Skip to content

Conversation

@Boog900
Copy link
Contributor

@Boog900 Boog900 commented Nov 18, 2025

Pull Request

What does this PR do?

Adds a method to get a value from a (key, value). Although on the surface this may seem useless, if you define the values comparison function to only take into account part of the value then you can use this method to do direct lookups of multi-map values.

This method would probably be less necessary if we exposed Cursors, which I would like, however this is a much smaller change and fits my use-case.

@Boog900 Boog900 marked this pull request as draft November 19, 2025 18:19
@Boog900 Boog900 marked this pull request as ready for review November 19, 2025 19:08
@Kerollmops
Copy link
Member

Hey @Boog900 👋

Thank you for the change, however, I'm wondering if we could instead work on exposing the internal Cursor I use in the Iterators.

What do you think?

@Boog900
Copy link
Contributor Author

Boog900 commented Nov 20, 2025

I would support that and it would allow me to do more things to make my code more efficient, I added this method as I didn't want to make too much of a change to the API and it was the least I needed to get my code working.

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