- Notifications
You must be signed in to change notification settings - Fork44
Add API to correct defective combining character sequences#98
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
base:master
Are you sure you want to change the base?
Add API to correct defective combining character sequences#98
Uh oh!
There was an error while loading.Please reload this page.
Conversation
Manishearth left a comment
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.
Not convinced this is worth maintaining: it's new data and it's not exactly any algorithm from the standard, which is generally more work to maintain.
For new features I typically would like to see stronger justification why this crate should support them
Jules-Bertholet commentedMar 4, 2024 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
My motivation here is that this API pairs well width |
b83714b tod4decaeCompareManishearth commentedMar 4, 2024
In that case please include some description of this API's use cases in its documentation. I still don't know if or when I'll be able to review this and the other PRs. |
4347629 to2dbe07fCompare
No description provided.