code review 5372107: sql: document that for drivers, io.EOF means no more rows

sql: document that for drivers, io.EOF means no more rows

This was used in the sql package + tests, but never
documented.

[bradfitz, 2011-11-15 22:23:40]

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

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

[gri, 2011-11-15 22:28:51]

LGTM

On Tue, Nov 15, 2011 at 2:23 PM,  <bradfitz@golang.org> wrote:
> Reviewers: golang-dev_googlegroups.com,
>
> Message:
> Hello golang-dev@googlegroups.com (cc: golang-dev@googlegroups.com),
>
> I'd like you to review this change to
> https://go.googlecode.com/hg/
>
>
> Description:
> sql: document that for drivers, io.EOF means no more rows
>
> This was used in the sql package + tests, but never
> documented.
>
> Please review this at http://codereview.appspot.com/5372107/
>
> Affected files:
>  M src/pkg/exp/sql/driver/driver.go
>
>
> Index: src/pkg/exp/sql/driver/driver.go
> ===================================================================
> --- a/src/pkg/exp/sql/driver/driver.go
> +++ b/src/pkg/exp/sql/driver/driver.go
> @@ -138,6 +138,8 @@
>        // The dest slice may be populated with only with values
>        // of subset types defined above, but excluding string.
>        // All string values must be converted to []byte.
> +       //
> +       // Next should return io.EOF when there are no more rows.
>        Next(dest []interface{}) error
>  }
>
>
>
>

[bradfitz, 2011-11-15 22:29:52]

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

sql: document that for drivers, io.EOF means no more rows

This was used in the sql package + tests, but never
documented.

R=golang-dev, gri
CC=golang-dev
http://codereview.appspot.com/5372107