]> Cypherpunks repositories - gostls13.git/commit
database/sql/driver: remove string exclusion
authorTamir Duberstein <tamird@gmail.com>
Fri, 12 Feb 2016 05:37:52 +0000 (00:37 -0500)
committerBrad Fitzpatrick <bradfitz@golang.org>
Wed, 23 Mar 2016 02:42:31 +0000 (02:42 +0000)
commit7162c4d05c1fed9db517c92b9a5232243ec453a2
tree9aaa8102fe6c17a9d731a42fa4137defa07445a5
parentbac0005ec77994b093c51e29f630125130c99b98
database/sql/driver: remove string exclusion

The exclusion of string from IsScanValue prevents driver authors from
writing their drivers in such a way that would allow users to
distinguish between strings and byte arrays returned from a database.
Such drivers are possible today, but require their authors to deviate
from the guidance provided by the standard library.

This exclusion has been in place since the birth of this package in
https://github.com/golang/go/commit/357f2cb1a385f4d1418e48856f9abe0cce,
but the fakedb implementation shipped in the same commit violates the
exclusion!

Strictly speaking this is a breaking change, but it increases the set
of permissible Scan types, and should not cause breakage in practice.

No test changes are necessary because fakedb already exercises this.

Fixes #6497.

Change-Id: I69dbd3a59d90464bcae8c852d7ec6c97bfd120f8
Reviewed-on: https://go-review.googlesource.com/19439
Run-TryBot: Brad Fitzpatrick <bradfitz@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
src/database/sql/driver/driver.go
src/database/sql/driver/types.go