]> Cypherpunks repositories - gostls13.git/commitdiff
math/big: correct umax
authorRobert Griesemer <gri@golang.org>
Mon, 9 Feb 2015 19:26:56 +0000 (11:26 -0800)
committerRobert Griesemer <gri@golang.org>
Mon, 9 Feb 2015 19:32:12 +0000 (19:32 +0000)
Change-Id: I208c8ac44d1a8882d8fdeb18347dc20941e20374
Reviewed-on: https://go-review.googlesource.com/4250
Reviewed-by: Alan Donovan <adonovan@google.com>
src/math/big/float.go

index f49d5b2fe5a1f1e5660dbe01721472d85a74e82e..d280916d0dba08d1d196c8af71a79fb33b511d81 100644 (file)
@@ -439,7 +439,7 @@ func (z *Float) SetInt64(x int64) *Float {
        return z
 }
 
-// SetInt64 sets z to the (possibly rounded) value of x and returns z.
+// SetFloat64 sets z to the (possibly rounded) value of x and returns z.
 // If z's precision is 0, it is changed to 53 (and rounding will have
 // no effect).
 // If x is denormalized or NaN, the result is unspecified.
@@ -525,8 +525,7 @@ func (z *Float) SetRat(x *Rat) *Float {
        a.SetInt(x.Num())
        b.SetInt(x.Denom())
        if z.prec == 0 {
-               // TODO(gri) think about a.prec type to avoid excessive conversions
-               z.prec = uint(max(int(a.prec), int(b.prec)))
+               z.prec = umax(a.prec, b.prec)
        }
        return z.Quo(&a, &b)
 }
@@ -1072,7 +1071,7 @@ func (x *Float) Sign() int {
 }
 
 func umax(x, y uint) uint {
-       if x < y {
+       if x > y {
                return x
        }
        return y