code review 5581050: godoc: sort list of "other packages"

godoc: sort list of "other packages"

Fixes issue 2786.

[gri, 2012-01-30 22:27:21]

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

I'd like you to review this change to
https://code.google.com/p/go

[bradfitz, 2012-01-30 22:30:17]

LGTM

On Mon, Jan 30, 2012 at 2:27 PM, <gri@golang.org> wrote:

> Reviewers: r,
>
> Message:
> Hello r (cc: golang-dev@googlegroups.com),
>
> I'd like you to review this change to
> https://code.google.com/p/go
>
>
> Description:
> godoc: sort list of "other packages"
>
> Fixes issue 2786.
>
> Please review this at
http://codereview.appspot.com/**5581050/<http://codereview.appspot.com/5581050/>
>
> Affected files:
>  M src/cmd/godoc/godoc.go
>
>
> Index: src/cmd/godoc/godoc.go
> ==============================**==============================**=======
> --- a/src/cmd/godoc/godoc.go
> +++ b/src/cmd/godoc/godoc.go
> @@ -1065,6 +1065,7 @@
>                        }
>                }
>                plist = plist[0:i]
> +               sort.Strings(plist)
>        }
>
>        // get examples from *_test.go files
>
>
>

[gri, 2012-01-30 22:31:53]

*** Submitted as dc7f79145d28 ***

godoc: sort list of "other packages"

Fixes issue 2786.

R=r, bradfitz
CC=golang-dev
http://codereview.appspot.com/5581050