-
Notifications
You must be signed in to change notification settings - Fork 226
Open
Labels
C-datetimeComponent: datetime, calendars, time zonesComponent: datetime, calendars, time zones
Milestone
Description
Currently we hardcode some glue patterns as just being space-separated
icu4x/provider/source/src/datetime/neo.rs
Lines 547 to 554 in 6c14cef
| GlueType::DateZone => { | |
| // TODO: Use proper appendItem here | |
| "{1} {2}" | |
| } | |
| GlueType::TimeZone => { | |
| // TODO: Use proper appendItem here | |
| "{0} {2}" | |
| } |
This is, most of the time, the right thing, but there are presumably cases where it isn't and we should use the timezone appendItem pattern instead:
This is specced in https://unicode.org/reports/tr35/tr35-dates.html#Missing_Skeleton_Fields
Metadata
Metadata
Assignees
Labels
C-datetimeComponent: datetime, calendars, time zonesComponent: datetime, calendars, time zones