D issues are now tracked on GitHub. This Bugzilla instance remains as a read-only archive.
Issue 23540 - std.uni loadProperty aliases for C are incorrect
Summary: std.uni loadProperty aliases for C are incorrect
Status: RESOLVED FIXED
Alias: None
Product: D
Classification: Unclassified
Component: phobos (show other issues)
Version: D2
Hardware: x86 All
: P3 normal
Assignee: No Owner
URL:
Keywords: pull
Depends on:
Blocks:
 
Reported: 2022-12-06 11:43 UTC by Richard (Rikki) Andrew Cattermole
Modified: 2022-12-24 12:39 UTC (History)
0 users

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this issue.
Description Richard (Rikki) Andrew Cattermole 2022-12-06 11:43:53 UTC 
In std.uni the function loadProperty handles aliases as defined by TR44, the current values of C do not match the values defined by TR44.

They should be: Cc | Cf | Cs | Co | Cn.

https://www.unicode.org/reports/tr44/#Property_Values

This is a breaking change. I want to hold off fixing this until my PR goes in for the table update due to the potential of that breaking peoples code as well. Best the breakage happens all in one release rather than spread out. However, it would be nice to know why it was done like this originally.

https://github.com/dlang/phobos/pull/8617
Comment 1 Dlang Bot 2022-12-19 19:30:41 UTC 
@rikkimax created dlang/phobos pull request #8652 "Fix issue 23540 - std.uni loadProperty aliases for C are incorrect" fixing this issue:

- Fix issue 23540 - std.uni loadProperty aliases for C are incorrect

https://github.com/dlang/phobos/pull/8652
Comment 2 Dlang Bot 2022-12-24 12:39:36 UTC 
dlang/phobos pull request #8652 "Fix issue 23540 - std.uni loadProperty aliases for C are incorrect" was merged into master:

- 13ef27a56e4c22e122fc4dd54bb46b5955babdb0 by richard (rikki) andrew cattermole:
  Fix issue 23540 - std.uni loadProperty aliases for C are incorrect

https://github.com/dlang/phobos/pull/8652