LGTM
On Mar 11, 2010, at 4:37 PM, gri@golang.org wrote:
> Reviewers: r,
>
> Message:
> Hello r (cc: golang-dev@googlegroups.com),
>
> I'd like you to review this change.
>
>
> Description:
> math: minor comment fixes
>
> Please review this at http://codereview.appspot.com/444043/show
>
> Affected files:
> M src/pkg/math/ldexp.go
> M src/pkg/path/path.go
>
>
> Index: src/pkg/math/ldexp.go
> ===================================================================
> --- a/src/pkg/math/ldexp.go
> +++ b/src/pkg/math/ldexp.go
> @@ -5,7 +5,7 @@
> package math
>
> // Ldexp is the inverse of Frexp.
> -// It returns frac × 2<sup>exp</sup>.
> +// It returns frac × 2^exp.
> func Ldexp(frac float64, exp int) float64 {
> // TODO(rsc): Remove manual inlining of IsNaN, IsInf
> // when compiler does it for us
> Index: src/pkg/path/path.go
> ===================================================================
> --- a/src/pkg/path/path.go
> +++ b/src/pkg/path/path.go
> @@ -115,7 +115,7 @@
> return "", path
> }
>
> -// Join joins any number of path elemets into a single path, adding a
> +// Join joins any number of path elements into a single path,
> adding a
> // separating slash if necessary. All empty strings are ignored.
> func Join(elem ...string) string {
> for i, e := range elem {
>
>
Issue 444043: code review 444043: math: minor comment fixes
(Closed)
Created 15 years ago by gri
Modified 15 years ago
Reviewers:
Base URL:
Comments: 0