unicodetools icon indicating copy to clipboard operation
unicodetools copied to clipboard

14 letters with palatal hook

Open eggrobin opened this issue 1 year ago • 2 comments

L2/24-050, Unicode request for letters with palatal hook. See https://github.com/unicode-org/sah/issues/350.

UTC-179-C59 Consensus: Provisionally assign 14 Latin code points U+1DF2D..U+1DF3A for letters with palatal hooks in Latin Extended-G block as described in document L2/24-050. [Ref. Section 14 of document L2/24-068]

eggrobin avatar Feb 17 '24 00:02 eggrobin

Please add the DoNotEmit data too.

Done. If we are going to get much more of these kinds of proposals, we need to find a way to incorporate validation and regeneration of DoNotEmit.txt in the tools; sorting these entries by hand is error-prone, and having data that is not validated is a good way to have bugs (I could easily have missed https://github.com/unicode-org/sah/issues/350#issuecomment-2018043770).

(The comments also want to be generated; I did not check them, and we should not have to check them by hand.)

eggrobin avatar Mar 25 '24 13:03 eggrobin

If we are going to get much more of these kinds of proposals, we need to find a way to incorporate validation and regeneration of DoNotEmit.txt in the tools; sorting these entries by hand is error-prone, and having data that is not validated is a good way to have bugs (I could easily have missed unicode-org/sah#350 (comment)).

I agree on principle, but I don't understand unicodetools enough to contribute to tooling.

Code reviewers are supposed to catch these issues.

(The comments also want to be generated; I did not check them, and we should not have to check them by hand.)

I have a Python script that deletes the character name comments and regenerates them. I can rerun it after these are merged, to double check.

roozbehp avatar Mar 26 '24 03:03 roozbehp