Branch: refs/heads/main
Home: https://github.com/WebKit/WebKit
Commit: 09fe0b34ffb6b1f0b547f7f6e4b9e3f4f0da54ba
https://github.com/WebKit/WebKit/commit/09fe0b34ffb6b1f0b547f7f6e4b9e3f4f0da54ba
Author: Tim Chevalier <[email protected]>
Date: 2025-04-15 (Tue, 15 Apr 2025)
Changed paths:
M JSTests/stress/temporal-instant.js
M JSTests/test262/config.yaml
M Source/JavaScriptCore/runtime/ISO8601.cpp
Log Message:
-----------
[JSC] Handle multiple Temporal calendar annotations
https://bugs.webkit.org/show_bug.cgi?id=223166
Reviewed by Justin Michaud.
This is part of a normative change to the Temporal proposal:
https://github.com/tc39/proposal-temporal/pull/2397
This change allows parsing multiple annotations in Temporal strings. If
there are multiple `u-ca` calendar annotations, the first one is used.
But if there are multiple calendars and any have the `!` flag, the
string is rejected because that's inconsistent.
This makes a lot of test262 tests newly pass and they can be removed
from the skip list. Some other tests under the same heading still fail
because PlainDate and PlainDateTime don't yet support calendars, so
recategorize them under the "Depends on calendar support" heading.
* JSTests/stress/temporal-instant.js:
* JSTests/test262/config.yaml:
* Source/JavaScriptCore/runtime/ISO8601.cpp:
(JSC::ISO8601::parseOneCalendar):
(JSC::ISO8601::parseCalendar):
(JSC::ISO8601::parseCalendarTime):
(JSC::ISO8601::parseCalendarDateTime):
Canonical link: https://commits.webkit.org/293742@main
To unsubscribe from these emails, change your notification settings at
https://github.com/WebKit/WebKit/settings/notifications
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
