code review 1004045: gmp: fix bug in SetString

gmp: fix bug in SetString

[rsc, 2010-05-01 00:32:14]

Hello adg (cc: golang-dev@googlegroups.com),

I'd like you to review this change.

[adg, 2010-05-01 02:42:43]

LGTM

On 1 May 2010 10:32,  <rsc@golang.org> wrote:
> Reviewers: adg,
>
> Message:
> Hello adg (cc: golang-dev@googlegroups.com),
>
> I'd like you to review this change.
>
>
> Description:
> gmp: fix bug in SetString
>
> Please review this at http://codereview.appspot.com/1004045/show
>
> Affected files:
>  M misc/cgo/gmp/gmp.go
>
>
> Index: misc/cgo/gmp/gmp.go
> ===================================================================
> --- a/misc/cgo/gmp/gmp.go
> +++ b/misc/cgo/gmp/gmp.go
> @@ -190,7 +190,7 @@
>        if C.mpz_set_str(&z.i[0], p, C.int(base)) < 0 {
>                return os.EINVAL
>        }
> -       return z
> +       return nil
>  }
>
>  // String returns the decimal representation of z.
>
>
>

[rsc, 2010-05-01 20:10:04]

*** Submitted as http://code.google.com/p/go/source/detail?r=ab26c319a006 ***

gmp: fix bug in SetString

R=adg
CC=golang-dev
http://codereview.appspot.com/1004045