tc39 / proposal-canonical-tz

TC39 Proposal (stacked on Temporal) to improve handling of changes to the IANA Time Zone Database
Other
38 stars 2 forks source link

Temporal.TimeZone.prototype.equals: Missing call to ToTemporalTimeZoneSlotValue #43

Closed anba closed 11 months ago

anba commented 11 months ago

Step 3:

Return ? TimeZoneEquals(timeZone, other).

But other first needs to be converted to a time-zone value through ToTemporalTimeZoneSlotValue.

ptomato commented 11 months ago

Another good find. This was introduced in adbdbf1d meaning it's actually part of the Time Zone Canonicalization proposal, which, since the July TC39 meeting, now lives inside the Temporal proposal.

I think we should just fix it since it seems pretty clear what the intention was, but I'll punt it over to proposal-canonical-tz for now since I'm not actually a champion of that proposal.

ptomato commented 11 months ago

(For some reason I cannot transfer it, so I'll copy it manually.)

ljharb commented 11 months ago

→ #41