-
-
Notifications
You must be signed in to change notification settings - Fork 44
Ucdxml 17v1 #1104
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
base: main
Are you sure you want to change the base?
Ucdxml 17v1 #1104
Conversation
jowilco
commented
Apr 25, 2025
•
edited
Loading
edited
- Addresses https://www.unicode.org/L2/L2019/19270.htm#160-A10 by consuming the RegEx syntax for Unihan directly from UAX38.
- Resolved https://www.unicode.org/L2/L2024/24221.htm#181-A43 by removing kGB7.
- Resolved https://www.unicode.org/L2/L2024/24221.htm#181-A131 by updating the RegEx for the na attribute.
- Addresses https://www.unicode.org/L2/L2024/24221.htm#181-A133 by adding support for Unicode Version 17.0
- Resolved https://www.unicode.org/L2/L2025/25085.htm#183-A100 by removing kJa.
- Resolved https://www.unicode.org/L2/L2025/25085.htm#183-A127 by adding kTayNumeric.
- Resolved https://www.unicode.org/L2/L2025/25085.htm#183-A180. All Unihan syntaxes are sourced directly from UAX38.
- Addresses UCDXML: add Unikemet properties #921 by adding support for Unikemet.
- Resolved Update UAX42 to document that common Unihan attributes are grouped starting with Unicode 17 #1071 by adding a comment indicating that "Unihan attributes are applied at the group where applicable, similar to how non-Unihan attributes are applied at the group."
- Partially addresses Update UAX42 to document which UCDXML fields correspond to UCD properties (UAX44) vs. which are “just data” corresponding to various UCD files #1049 by removing Deprecated properties, Normalization Corrections, and Emoji Sources.
…istorical data, and added partial support for kEH_AltSeq
Hi @jowilco the CI failures suggest that you need to run GenerateEnums again, and also |
Agreed. This is still a draft pending the fixes for TR57. I'll definitely clean everything up before removing the draft status. |
//Deprecated | ||
// createPropertyFragment( | ||
// UcdProperty.ISO_Comment, | ||
// SCHEMA.PROPERTIES, | ||
// getFormattedSyntax(UcdProperty.ISO_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.
i suggest you just delete this altogether
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.
Done
//Deprecated | ||
// case FC_NFKC_Closure: |
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.
i suggest you just delete this altogether
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.
Done.
//Deprecated | ||
// + TRIPLELINE | ||
// + getFormattedBoolean(UcdProperty.Expands_On_NFC) | ||
// + DOUBLELINE | ||
// + getFormattedBoolean(UcdProperty.Expands_On_NFD) | ||
// + DOUBLELINE | ||
// + getFormattedBoolean(UcdProperty.Expands_On_NFKC) | ||
// + DOUBLELINE | ||
// + getFormattedBoolean(UcdProperty.Expands_On_NFKD) | ||
// + TRIPLELINE | ||
// + getFormattedSyntax(UcdProperty.FC_NFKC_Closure); |
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.
i suggest you just delete this altogether
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.
Done
//Deprecated | ||
// + getFormattedBoolean(UcdProperty.Hyphen) | ||
// + DOUBLELINE |
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.
ditto
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.
Done
//Deprecated | ||
// + getFormattedBoolean(UcdProperty.Grapheme_Link) | ||
// + DOUBLELINE |
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.
ditto
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.
Done
// Deprecated | ||
// public static UCDPropertyDetail Hyphen_Detail = | ||
// new UCDPropertyDetail( | ||
// UcdProperty.Hyphen, | ||
// VersionInfo.getInstance(2, 0, 0), | ||
// 32, | ||
// true, | ||
// false, | ||
// false, | ||
// true, | ||
// false); |
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.
i suggest you just delete this altogether
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.
I screwed up here - I should have added values for maxVersion, which I have done now.
// Deprecated | ||
// public static UCDPropertyDetail Grapheme_Link_Detail = | ||
// new UCDPropertyDetail( | ||
// UcdProperty.Grapheme_Link, | ||
// VersionInfo.getInstance(3, 2, 0), | ||
// 59, | ||
// true, | ||
// false, | ||
// false, | ||
// true, | ||
// false); |
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.
i suggest you just delete this altogether
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.
I screwed up here - I should have added values for maxVersion, which I have done now.
true); | ||
public static UCDPropertyDetail Expands_On_NFD_Detail = | ||
false); | ||
// Deprecated |
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.
i suggest you just delete this altogether
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.
I screwed up here - I should have added values for maxVersion, which I have done now.
false, | ||
true); | ||
false); | ||
// public static UCDPropertyDetail kGB7_Detail = |
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.
i suggest you just delete this altogether
false, | ||
true); | ||
false); | ||
// Deprecated |
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.
i suggest you just delete this altogether
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.
I screwed up here - I should have added values for maxVersion, which I have done now.
…istorical data, and added partial support for kEH_AltSeq