Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

math/big: add substantial documentation #11241

Closed
adg opened this issue Jun 16, 2015 · 7 comments
Closed

math/big: add substantial documentation #11241

adg opened this issue Jun 16, 2015 · 7 comments
Milestone

Comments

@adg
Copy link
Contributor

adg commented Jun 16, 2015

The math/big package is mysterious to newcomers. It's even a bit mysterious to seasoned Go programmers. We'd do well to include some large examples, and expand a bit in the package comment and the doc comments of Rat and Int.

cc @griesemer

@ALTree
Copy link
Member

ALTree commented Jun 17, 2015

Something like this? (as a first little step)

https://go-review.googlesource.com/#/c/11034

@gopherbot
Copy link

CL https://golang.org/cl/11034 mentions this issue.

griesemer pushed a commit that referenced this issue Jun 17, 2015
Updates #11241

Change-Id: I9639c4f66cf805a57b087c9f648d3918df105d86
Reviewed-on: https://go-review.googlesource.com/11034
Reviewed-by: Robert Griesemer <gri@golang.org>
@gopherbot
Copy link

CL https://golang.org/cl/11245 mentions this issue.

griesemer pushed a commit that referenced this issue Jun 18, 2015
Updates #11241

Change-Id: I573be85d0cfcf410f6125ecd2be8a3d292c40bbb
Reviewed-on: https://go-review.googlesource.com/11245
Reviewed-by: Robert Griesemer <gri@golang.org>
@ianlancetaylor ianlancetaylor added this to the Go1.6 milestone Jul 11, 2015
@gopherbot
Copy link

CL https://golang.org/cl/14317 mentions this issue.

@gopherbot
Copy link

CL https://golang.org/cl/14356 mentions this issue.

mk0x9 pushed a commit to mk0x9/go that referenced this issue Sep 10, 2015
Updates golang#11241

Change-Id: I0614c5a9a7a4c399ad5d664f36c70c3210911905
Reviewed-on: https://go-review.googlesource.com/14356
Reviewed-by: Robert Griesemer <gri@golang.org>
griesemer pushed a commit that referenced this issue Sep 15, 2015
Updates #11241

Change-Id: If71f651f3b8aca432c91314358b93f195217d9ec
Reviewed-on: https://go-review.googlesource.com/14317
Reviewed-by: Robert Griesemer <gri@golang.org>
@gopherbot
Copy link

CL https://golang.org/cl/15042 mentions this issue.

griesemer added a commit that referenced this issue Sep 28, 2015
- moved existing package documentation from nat.go to doc.go
- expanded on it

For #11241.

Change-Id: Ie75a2b0178a8904a4154307a1f5080d7efc5489a
Reviewed-on: https://go-review.googlesource.com/15042
Reviewed-by: Alan Donovan <adonovan@google.com>
@gopherbot
Copy link

CL https://golang.org/cl/15111 mentions this issue.

@golang golang locked and limited conversation to collaborators Sep 28, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

5 participants