U+2B81E and Unihan 18 #1243
Merged
U+2B81E and Unihan 18 #1243
Conversation
This file contains hidden or 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
Member
eggrobin
commented
Nov 17, 2025
eggrobin
commented
- U+2B81E:
- [185-C19] Consensus: Accept the disunification of U+6B25 with the disunified G-source ideograph moved to a new code point U+2B81E at the end of the CJK Unified Ideographs Extension D block, based on Recommendation M65.06 in document IRG N2826 (IRG) and as amended in Section 26 of document L2/25-231, for Unicode Version 18.0.
- https://github.com/unicode-org/utc-release-management/issues/273
- Exeunt kIRGDaeJaweon and kIRGKangXi:
- [185-C14] Consensus: Accept the proposal to remove the provisional kIRGDaeJaweon and kIRGKangXi properties, based on document L2/25-214 and Section 18 of document L2/25-231, for Unicode Version 18.0.
eggrobin
added
data-for-new
pipeline-17.0
pipeline-18.0
and removed
pipeline-17.0
labels
roozbehp
reviewed
Nov 17, 2025
| if (ch <= 0x2B81D) { | ||
| return CJK_D_BASE; | ||
| } | ||
| if (ch <= 0x2B81E && rCompositeVersion >= 0x120000) { |
Contributor
There was a problem hiding this comment.
I don't understand rCompositeVersion. Otherwise LGTM.
Member
There was a problem hiding this comment.
It's the major version number in bits 23..16 (18=0x12), minor in 15..8, "milli" in 7..0.
Computing a single number makes range checks trivial compared with lexical comparison of multiple fields.
Member
Author
There was a problem hiding this comment.
Of course that number looked much more readable until Unicode 10…
markusicu
approved these changes
Nov 17, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
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.