-
Notifications
You must be signed in to change notification settings - Fork 5.4k
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
Minimal String type #4716
Minimal String type #4716
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, just some minor nits.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This makes a lot of sense over #4710
Create a minimal, forward compatible unicode string type that only accepts ASCII input and doesn't provide mutation of codepoints. This should provide Sway-Standards with a proper black box type to use while still giving us a good platform to later implement a proper, `char` based unicode string without compatibility issues.
ba7f8ca
to
e768c63
Compare
Description
Create a minimal, forward compatible unicode string type that only accepts ASCII input and doesn't provide mutation of codepoints.
This should provide Sway-Standards with a proper black box type to use while still giving us a good platform to later implement a proper,
char
based unicode string without compatibility issues.Follow-up from #4710
Checklist
Breaking*
orNew Feature
labels where relevant.