Skip to content

[NSArray] Add check to isEqual(to:) whether values is _ObjectBridgeable #990

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

Merged
merged 3 commits into from
May 23, 2017

Conversation

norio-nomura
Copy link
Contributor

@norio-nomura norio-nomura commented May 18, 2017

When NSArray are compared for equality, NSArray only checks if the element can be casted as AnyHashable. Therefore, the result was incorrect when the NSArray contained instances which are not Hashable but are subclasses of NSObject (eg NSArray and NSDictionary) .
This PR fixes that.

Fixes SR-4903

@norio-nomura
Copy link
Contributor Author

Updated description.

@alblue
Copy link
Contributor

alblue commented May 19, 2017

@swift-ci please test

@aciidgh
Copy link

aciidgh commented May 23, 2017

@parkera is this good to merge?

@parkera parkera merged commit 61a7760 into swiftlang:master May 23, 2017
@norio-nomura
Copy link
Contributor Author

Thanks! 🙏

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.

4 participants