Skip to content

Update to latest test262 and pull recent changes from upstream #338

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
May 15, 2025
Merged

Conversation

ptomato
Copy link
Contributor

@ptomato ptomato commented May 14, 2025

No description provided.

ptomato added 3 commits May 12, 2025 18:10
Instead of doing a multiplication here, we can just check if the
remainder is zero. This matters a bit more for TimeDuration and
RoundNumberToIncrementAsIfPositive, because we avoid a BigInt
multiplication.

UPSTREAM_COMMIT=407adc9e221eecb97e4db1c7b2b8e2b41cbea29a
For a few time zones, the two-week search window for an interval to
bisect is too large, because transitions occur less than two weeks
apart. Special-case these time zones.

This allows increasing the default window to 19 days (a bunch of
Argentinian time zones have transitions less than 20 days apart, so this
seems like a good boundary where we don't have to add too many special
cases.) This gives a small performance improvement in the common case.

UPSTREAM_COMMIT=da53e9acead66c3616971e487236bde1a731c564
This is currently unspecified behaviour, and is to be specified later in
tc39/proposal-intl-era-monthcode#15

Closes: #3096

UPSTREAM_COMMIT=2e156ecb102cd4e1c93b21b26f2611832f6499de
Copy link
Contributor

@justingrant justingrant left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

@ptomato ptomato merged commit 55619bc into main May 15, 2025
27 checks passed
@ptomato ptomato deleted the update branch May 15, 2025 18:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants