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

Keys are not equal when they have different codes and no value. #25

Merged
merged 1 commit into from
Dec 14, 2023

Conversation

Ambrevar
Copy link
Member

@Ambrevar Ambrevar commented Dec 12, 2023

Fixes #21.

Rather trivial, or am I missing something?

@Ambrevar Ambrevar merged commit 346eede into master Dec 14, 2023
1 check passed
@aadcg
Copy link
Member

aadcg commented Dec 14, 2023

In this case, I'd suggest adding a test that captures the invariant that has been addressed.

@Ambrevar
Copy link
Member Author

Good point, done with 0a06d79.

@aadcg
Copy link
Member

aadcg commented Dec 15, 2023

Great, exactly what I had in mind. Thanks!

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

Successfully merging this pull request may close these issues.

lookup-key returns wrong entry when looking up based on :code
2 participants