Skip to content

Fix serialization of durations with zero seconds and nonzero subseconds#610

Merged
nekevss merged 1 commit intoboa-dev:mainfrom
ptomato:fix-duration-serialization
Oct 30, 2025
Merged

Fix serialization of durations with zero seconds and nonzero subseconds#610
nekevss merged 1 commit intoboa-dev:mainfrom
ptomato:fix-duration-serialization

Conversation

@ptomato
Copy link
Copy Markdown
Contributor

@ptomato ptomato commented Oct 30, 2025

Previously, if a duration had nonzero minutes or higher, zero seconds, and nonzero subseconds, the subseconds would not be printed.

See test262 tests at tc39/test262#4620. Also adds simple inline tests for this case.

Previously, if a duration had nonzero minutes or higher, zero seconds,
and nonzero subseconds, the subseconds would not be printed.

See test262 tests at tc39/test262#4620. Also
adds simple inline tests for this case.
@ptomato ptomato force-pushed the fix-duration-serialization branch from c47b144 to 2c052c4 Compare October 30, 2025 00:58
Copy link
Copy Markdown
Member

@nekevss nekevss left a comment

Choose a reason for hiding this comment

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

Looks good!

Thanks!

@nekevss nekevss merged commit 02b37c7 into boa-dev:main Oct 30, 2025
8 checks passed
@ptomato ptomato deleted the fix-duration-serialization branch October 30, 2025 15:33
philwo pushed a commit to philwo/chromium that referenced this pull request Nov 6, 2025
Minor issue caught by snapshot testing.

Uplifts boa-dev/temporal#610

Bug: 401065166
Change-Id: Ia571f6bcac5a11264b69f5ffd9209c98eceecb2c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7127120
Reviewed-by: Łukasz Anforowicz <lukasza@chromium.org>
Auto-Submit: Manish Goregaokar <manishearth@google.com>
Commit-Queue: Manish Goregaokar <manishearth@google.com>
Commit-Queue: Łukasz Anforowicz <lukasza@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1541459}
mohd-akram pushed a commit to gsource-mirror/chromium-src-third_party-rust that referenced this pull request Jan 9, 2026
Minor issue caught by snapshot testing.

Uplifts boa-dev/temporal#610

Bug: 401065166
Change-Id: Ia571f6bcac5a11264b69f5ffd9209c98eceecb2c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7127120
Reviewed-by: Łukasz Anforowicz <lukasza@chromium.org>
Auto-Submit: Manish Goregaokar <manishearth@google.com>
Commit-Queue: Manish Goregaokar <manishearth@google.com>
Commit-Queue: Łukasz Anforowicz <lukasza@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1541459}
NOKEYCHECK=True
GitOrigin-RevId: c3e1f3b66b2b3a4a6a7262431311d165905ba40b
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