Skip to content
This repository has been archived by the owner on Sep 22, 2022. It is now read-only.

MDBX_SET_UPPERBOUND #251

Closed
wants to merge 2 commits into from
Closed

MDBX_SET_UPPERBOUND #251

wants to merge 2 commits into from

Conversation

gcxfd
Copy link
Contributor

@gcxfd gcxfd commented Dec 11, 2021

add MDBX_SET_UPPERBOUND

see issue #250

When the value is mdbx_val_empty(), the test result should be correct (the screenshot of the running result of my rust code is as below).

If there is a certain value, I am not sure whether the logic is correct, because I have not used this and I have not fully understood the code of MDBX_SET_LOWERBOUND, so please help me fix it.

@erthink
Copy link
Owner

erthink commented Dec 13, 2021

Basically upper bound is the same as lower bound, but should skip equal items.
So you solution have too much extra code, and unclear for me.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants