code review 4982041: godoc: fix bug in zip.go

godoc: fix bug in zip.go

The result of sort.Search is in the interval [0,n);
specifically, if no entry is found, the result is n
and not -1.

[gri, 2011-08-29 23:39:27]

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

I'd like you to review this change to
https://go.googlecode.com/hg/

[dsymonds, 2011-08-29 23:47:25]

LGTM

Seems like we should change sort.Search, though. It's an outlier when
compared to, for instance, strings.Index.

[gri, 2011-08-29 23:49:35]

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

godoc: fix bug in zip.go

The result of sort.Search is in the interval [0,n);
specifically, if no entry is found, the result is n
and not -1.

R=dsymonds
CC=golang-dev
http://codereview.appspot.com/4982041

[rsc, 2011-08-29 23:52:16]

On Mon, Aug 29, 2011 at 19:47, David Symonds <dsymonds@golang.org> wrote:
> Seems like we should change sort.Search, though. It's an outlier when
> compared to, for instance, strings.Index.

sort.Search is different from strings.Index.
It tells you where the element would be
if it were in the array.  No element would
ever be at index -1.

Russ