-
Notifications
You must be signed in to change notification settings - Fork 23
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
Do we need to discuss regular expressions? #54
Comments
We need to consider this question in WG. |
Discussed in teleconference https://www.w3.org/2017/06/01-i18n-minutes.html The notes there are not very helpful, since we didn't record the conversation, but changes to spec to follow. |
Currently we have this text in 5.1:
Is that enough to close? Do we need a section discussing regex? |
"Unicode encoding variations" is not a defined term, and if people look up variation, they will only find standardized variation sequences, which I am sure were not of uppermost concerns here. I can't tell (even after looking at the original in more detail) whether the concern about "variation" was about encoding forms or normalization forms. The header of the section mentions normalization, but encoding forms are also discussed. It may be worth noting that in some cases comparisons should be preferably done in NFD - this is the case for comparing domain names against confusable variants, to give one example. |
It's about the various and sundry different ways text can be encoded. It's not meant to be a term. That paragraph should be made clearer. |
@asmusf check the above edit and see if that works better. Suggest edits as needed. |
Definitely better. Source code for the text is now a single very long line per paragraph, just pointing that out if it matters. |
Closing this issue. Reopen if needed |
There is an issue comment in the current text that says:
Here is the proposed requirement (also in the current text):
Should we add more discussion of regular expressions? Is that beyond the scope of our document? Should we keep the above requirement? Or should we do something different?
The text was updated successfully, but these errors were encountered: