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

Equality for unordered_set and unordered_map should be based on equality of elements #4406

Merged
merged 10 commits into from
Feb 27, 2024

Conversation

fsb4000
Copy link
Contributor

@fsb4000 fsb4000 commented Feb 18, 2024

Fixes #4388

@fsb4000 fsb4000 requested a review from a team as a code owner February 18, 2024 08:57
@StephanTLavavej StephanTLavavej added the bug Something isn't working label Feb 18, 2024
@StephanTLavavej StephanTLavavej self-assigned this Feb 18, 2024
tests/std/test.lst Outdated Show resolved Hide resolved
@StephanTLavavej
Copy link
Member

Thanks for fixing this long-standing bug (and removing my confused _Nonkfn machinery)! I pushed some changes to the test.

@StephanTLavavej StephanTLavavej removed their assignment Feb 22, 2024
@StephanTLavavej StephanTLavavej self-assigned this Feb 23, 2024
@StephanTLavavej
Copy link
Member

I'm mirroring this to the MSVC-internal repo - please notify me if any further changes are pushed.

@StephanTLavavej StephanTLavavej merged commit e57fc4f into microsoft:main Feb 27, 2024
35 checks passed
@StephanTLavavej
Copy link
Member

Thanks for fixing this long-standing bug! 🐞 🛠️ 🎉

@fsb4000 fsb4000 deleted the fix4388 branch February 27, 2024 14:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

<unordered_set>, <unordered_map>: operator== is incorrect with custom equivalence functor
3 participants