fyne-io / fyne / 42457847951

62%

develop: 62%

Pull #6664

github

Return source read errors from IntToFloat.Set

Motivation: IntToFloat(src).Set(v) compared the truncated value with
the result of src.Get() before checking Get's error. When Get returned
(0, err) and v truncated to 0 (for example 0, 0.75 or -0.75), Set
returned nil and the read error was dropped.

Approach: check the error from Get before comparing old and new values,
matching the order already used by toInt.Set. No other behaviour changes.

Validation: added TestIntToFloat_SetReturnsSourceReadError. Without the
fix it fails (Set returns nil for 0, 0.75 and -0.75); with the fix
`go test ./data/binding` passes.

Report: https://github.com/fyne-io/fyne/issues/6663
Signed-off-by: Pujitha Paladugu <10557236+pujitha24@users.noreply.github.com>
Assisted-by: claude-sonnet-5-5 (via Claude Code)

1 of 3 new or added lines in 1 file covered. (33.33%)

26081 of 42319 relevant lines covered (61.63%)

16159.09 hits per line