Skip to content

Conversation

stanim
Copy link
Contributor

@stanim stanim commented Apr 14, 2016

Any error encountered during iteration should be handled properly, see:
https://golang.org/pkg/database/sql/#Rows
http://go-database-sql.org/retrieving.html

@stanim stanim changed the title check for an error at the end of the for rows.Next() loop add missing error checks in simple example Apr 14, 2016
@coveralls
Copy link

Coverage Status

Coverage remained the same at 68.137% when pulling ed21175 on stanim:errs into 37aa7c6 on mattn:master.

@mattn mattn merged commit aeb3180 into mattn:master Apr 15, 2016
@mattn
Copy link
Owner

mattn commented Apr 15, 2016

thank you

@stanim stanim deleted the errs branch April 15, 2016 08:45
@stanim
Copy link
Contributor Author

stanim commented Apr 15, 2016

You're welcome.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants