]> Cypherpunks repositories - gostls13.git/commit
math/big: remove NaN support - just not worth it
authorRobert Griesemer <gri@golang.org>
Tue, 31 Mar 2015 01:11:48 +0000 (18:11 -0700)
committerRobert Griesemer <gri@golang.org>
Tue, 31 Mar 2015 23:05:31 +0000 (23:05 +0000)
commitfa85a7206d46dc30d59cfd4e882b58a59a0a3d88
tree1d46eee25ee0a710c37e8302b657421775b13dd4
parent67426a8a9eabd6d859e42ca799eab6c1aa0d616a
math/big: remove NaN support - just not worth it

NaNs make the API more complicated for no real good reasons.
There are few operations that produce NaNs with IEEE arithmetic,
there's no need to copy the behavior. It's easy to test for these
scenarios and avoid them (on the other hand, it's not easy to test
for overflow or underflow, so we want to keep +/-Inf).

Also:
- renamed IsNeg -> Signbit (clearer, especially for x == -0)
- removed IsZero           (Sign() == 0 is sufficient and efficient)
- removed IsFinite         (now same as !IsInf)

Change-Id: I3f3b4445c325d9bbb1bf46ce2e298a6aeb498e07
Reviewed-on: https://go-review.googlesource.com/8280
Reviewed-by: Alan Donovan <adonovan@google.com>
src/math/big/accuracy_string.go
src/math/big/float.go
src/math/big/float_test.go
src/math/big/floatconv.go
src/math/big/floatconv_test.go
src/math/big/floatexample_test.go
src/math/big/ftoa.go