Skip to content

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

Merged
StephanTLavavej merged 10 commits intomicrosoft:mainfrom
fsb4000:fix4388
Feb 27, 2024
Merged

Equality for unordered_set and unordered_map should be based on equality of elements#4406
StephanTLavavej merged 10 commits intomicrosoft:mainfrom
fsb4000:fix4388

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
@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
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
@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

None yet

Development

Successfully merging this pull request may close these issues.

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

3 participants