-
-
Notifications
You must be signed in to change notification settings - Fork 33
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
Semicolons; revisited #819
Conversation
reduce set of semicolons in SContinue to include only U+003B and its compatibility equivalents. Spurred by https://www.unicode.org/review/pri494/feedback.html#ID20240509103608 and discussion amongst PAG members.
re-generated data
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 o.k.
Hi @josh-hadley are you ready to merge this? I would like to get a snapshot of the UCD proper for UCDXML and the beta. |
Almost ready. I wanted to give the author of the feedback that initiated this change an opportunity to look it over first. If I don't hear anything soon I'll go ahead and merge and get you un-blocked. |
My original feedback was specifically about the non-common-script semicolons. This PR also removes two common-script semicolons: U+204F REVERSED SEMICOLON and U+2E35 TURNED SEMICOLON. I don’t know how those should be categorized, but I wanted to check with you that their removal was intentional. FWIW, L2/12-168 shows U+204F REVERSED SEMICOLON used as a standard semicolon in right-to-left Old Hungarian. That’s all the feedback I have on this PR. |
Thanks, David. |
Agreed, and I'll proceed with that. Thanks for responding @dscorbett! |
pre-16.0 beta public feedback pointed out some issues with the first pass at addressing UTC-179-A113. After some discussion with other PAG members, we decided to make the definition of "Semicolons" to be U+003B and its compatibility equivalents.