diff options
| author | Daniel Theophanes <kardianos@gmail.com> | 2016-11-23 09:10:30 -0800 |
|---|---|---|
| committer | Brad Fitzpatrick <bradfitz@golang.org> | 2016-12-01 16:52:12 +0000 |
| commit | e5e05627745764fb9989bf3966919d6715f21abc (patch) | |
| tree | a504ad49f44371aab7d2b62eeb92108e6470df5d /src/database/sql/fakedb_test.go | |
| parent | ea1b90f855fe2891ff0f39d3a3f4b0a5f72a04ed (diff) | |
| download | go-e5e05627745764fb9989bf3966919d6715f21abc.tar.xz | |
database/sql: document expectations for named parameters
Require parameter names to not begin with a symbol.
Change-Id: I5dfe9d4e181f0daf71dad2f395aca41c68678cbe
Reviewed-on: https://go-review.googlesource.com/33493
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
Run-TryBot: Brad Fitzpatrick <bradfitz@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Diffstat (limited to 'src/database/sql/fakedb_test.go')
| -rw-r--r-- | src/database/sql/fakedb_test.go | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/database/sql/fakedb_test.go b/src/database/sql/fakedb_test.go index 416b97d501..4b15f5bec7 100644 --- a/src/database/sql/fakedb_test.go +++ b/src/database/sql/fakedb_test.go @@ -713,7 +713,7 @@ func (s *fakeStmt) execInsert(args []driver.NamedValue, doInsert bool) (driver.R } else { // Assign value from argument placeholder name. for _, a := range args { - if a.Name == strvalue { + if a.Name == strvalue[1:] { val = a.Value break } @@ -818,7 +818,7 @@ func (s *fakeStmt) QueryContext(ctx context.Context, args []driver.NamedValue) ( } else { // Assign arg value from placeholder name. for _, a := range args { - if a.Name == wcol.Placeholder { + if a.Name == wcol.Placeholder[1:] { argValue = a.Value break } |