Update to Unicode 13.0.0, and other packages #31
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
TimothyGu
commented
May 22, 2021
| // UTS #46 version 13.0.0 includes the following broken tests. | ||
| // They both include U+18C4E, which prior to 13.0.0 was treated as disallowed | ||
| // but became valid in 13.0.0. However, the IdnaTestV2.txt file does not appear | ||
| // to be updated for this change, and still considers them invalid. |
There was a problem hiding this comment.
Reported to Unicode:
## IdnaTestV2.txt issue
The IdnaTestV2.txt that came with UTS #46 Version 13.0.0 has a minor
error. ToUnicode for "xn--mbm8237g..xn--7-7hf" is marked as failure with
reason "V6", but this should not be the case. (This domain appears
_twice_ around line 3070.)
The cause appears to be the lack of update for version 13.0.0. The
problematic domain contains the code point U+18C4E. While this code
point is "disallowed" in Unicode 12.0.0's IdnaMappingTable.txt (line
6397), it is "valid" under Unicode 13.0.0's (line 6450).
(Note, however, a similar domain "xn--mbm8237g.xn--7-7hf1526p" is
correct in the file and should remain forbidden. This is since it
additionally has the code point U+FE12, which is "disallowed" in both
12.0.0 and 13.0.0.)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.