Skip to content

[WIP][DNM] identical functions #2821

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

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

vanvoorden
Copy link

No description provided.

@vanvoorden
Copy link
Author

@lorentey Thanks for all the help so far! I'm going to continue working on this. My plan for right now is to actually split this up into two proposals:

  1. Add isIdentical functions where it makes sense in Standard Library and Foundation. This would include Array, Dictionary, Set, and other types where it makes sense.
  2. Add isKnownIdentical to Equatable with a default implementation returning nil.

I think it makes more sense to break these apart and review these independently. Mixed feedback on one does not have to block shipping the other. How does that idea sound?

I can plan to spend time writing out these two proposals and following up with questions in the review cycle. I can also plan to spend time writing the implementation code in standard library and foundation… but I might possibly have another friend I can recruit to also help for that.

My goal is to land before the 6.3 branch cut. I'm guessing that will be end of Q3.

You added a lot of great ideas and I would like to list you as a coauthor on both of these if you are interested. Let me know what you think. No super urgent rush… I'm blocked on something else for right now and I'm expecting to put some more time on this in about two or three weeks.

@rjmccall rjmccall added workgroup: needs implementation This proposal needs more implementation work before it can be reviewed LSG Contains topics under the domain of the Language Steering Group labels May 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
LSG Contains topics under the domain of the Language Steering Group workgroup: needs implementation This proposal needs more implementation work before it can be reviewed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants