Skip to content
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

揚琴 slow 2..4 #782

Draft
wants to merge 17 commits into
base: main
Choose a base branch
from
Draft

揚琴 slow 2..4 #782

wants to merge 17 commits into from

Conversation

eggrobin
Copy link
Member

@eggrobin eggrobin commented Apr 23, 2024

See L2/24-071R, Proposal to encode three stable extended Suzhou Numeral–like letters for Cantonese Music, and the CJK report L2/24-067, Section 38).

See also

UTC-179-C44 Consensus: Accept the proposal to provisionally assign U+16FF4 YANGQIN SIGN SLOW TWO, U+16FF5 YANGQIN SIGN SLOW THREE, and U+16FF6 YANGQIN SIGN SLOW FOUR in the Ideographic Symbols and Punctuation block, based on document L2/24-071R2 (Chan) and Section 38 of document L2/24-067.

Copy link
Member

@markusicu markusicu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks like you need to either add these piecemeal to the $NonOtherLetterIdeographs, or else define that set as [\p{Ideographic} - \p{gc=Lo}] rather than testing that it's equal to that.

@eggrobin
Copy link
Member Author

It looks like you need to either add these piecemeal to the $NonOtherLetterIdeographs, or else define that set as [\p{Ideographic} - \p{gc=Lo}] rather than testing that it's equal to that.

I ended up doing both. It is a small enough set that there is some value, and not too much burden, in listing them.

Copy link
Member

@markusicu markusicu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

Copy link
Member

@markusicu markusicu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants