[go] math/big: fix edge case float formatting

0 views

Skip to first unread message

Neal Patel (Gerrit)

unread,

Jul 1, 2026, 1:52:51 PM (3 days ago) Jul 1

to Neal Patel, goph...@pubsubhelper.golang.org, golang-...@googlegroups.com, golang...@luci-project-accounts.iam.gserviceaccount.com, Robert Griesemer, Keith Randall, golang-co...@googlegroups.com

Neal Patel submitted the change

Unreviewed changes

13 is the latest approved patch-set.
No files were changed between the latest approved patch-set and the submitted one.

Change information

Commit message:

math/big: fix edge case float formatting

An edge case exists where index exceeds len(upper.mant)
due to decimal trailing-zero trimming; the new invariant
permits round-up past upper's mantissa when incrementing
the digit will not force a carry.

Fixes #80206

Change-Id: I9d346a7a67a7a6c3cfaea50ae6fb67331e853bcb

Reviewed-on: https://go-review.googlesource.com/c/go/+/779080

Reviewed-by: Neal Patel <neal...@google.com>

Reviewed-by: Robert Griesemer <g...@google.com>

LUCI-TryBot-Result: golang...@luci-project-accounts.iam.gserviceaccount.com <golang...@luci-project-accounts.iam.gserviceaccount.com>

Files:

M src/math/big/floatconv_test.go
M src/math/big/ftoa.go

Change size: S

Delta: 2 files changed, 29 insertions(+), 8 deletions(-)

Branch: refs/heads/master

Submit Requirements:

Code-Review: +1 by Neal Patel, +2 by Robert Griesemer
TryBots-Pass: LUCI-TryBot-Result+1 by golang...@luci-project-accounts.iam.gserviceaccount.com

Open in Gerrit

Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings.

Gerrit

open

diffy

satisfied_requirement

Reply all

Reply to author

Forward

0 new messages