From 32914bc90b7cda863506da40d997999f0c5da507 Mon Sep 17 00:00:00 2001 From: Philip Chimento Date: Mon, 16 May 2022 17:47:38 -0700 Subject: [PATCH 1/2] Normative: Accept and ignore calendar annotation in Instant string Parsing an Instant string already ignores the time zone name annotation if one is present, and only takes the offset into account. It's inconsistent to throw if a calendar annotation is present but not if a time zone name annotation is present. Closes: #2143 --- polyfill/test/validStrings.mjs | 2 +- spec/abstractops.html | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/polyfill/test/validStrings.mjs b/polyfill/test/validStrings.mjs index 984aba18f8..4d0a08fdf4 100644 --- a/polyfill/test/validStrings.mjs +++ b/polyfill/test/validStrings.mjs @@ -359,7 +359,7 @@ const duration = seq( choice(durationDate, durationTime) ); -const instant = seq(date, [timeSpecSeparator], timeZoneOffsetRequired); +const instant = seq(date, [timeSpecSeparator], timeZoneOffsetRequired, [calendar]); const zonedDateTime = seq(date, [timeSpecSeparator], timeZoneNameRequired, [calendar]); // goal elements diff --git a/spec/abstractops.html b/spec/abstractops.html index f0f456ef44..7fd43bf3f8 100644 --- a/spec/abstractops.html +++ b/spec/abstractops.html @@ -1186,7 +1186,7 @@

ISO 8601 grammar

Sign? DurationDesignator DurationTime TemporalInstantString : - Date TimeSpecSeparator? TimeZoneOffsetRequired + Date TimeSpecSeparator? TimeZoneOffsetRequired Calendar? TemporalDateTimeString : CalendarDateTime From b7cb9f486a31245cd86987021916f0d1d10a9df2 Mon Sep 17 00:00:00 2001 From: Philip Chimento Date: Thu, 25 Aug 2022 10:17:12 -0700 Subject: [PATCH 2/2] Update test262 --- polyfill/test262 | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/polyfill/test262 b/polyfill/test262 index 747bed2e8a..8dcc0e1955 160000 --- a/polyfill/test262 +++ b/polyfill/test262 @@ -1 +1 @@ -Subproject commit 747bed2e8aaafe8fdf2c65e8a10dd7ae64f66c47 +Subproject commit 8dcc0e1955b1753271ed0812d1a2a15a23de069b