]> Cypherpunks repositories - gostls13.git/commitdiff
database/sql: remove duplicate validation
authorTamir Duberstein <tamird@gmail.com>
Wed, 31 Jan 2018 16:58:03 +0000 (11:58 -0500)
committerDaniel Theophanes <kardianos@gmail.com>
Tue, 13 Feb 2018 21:25:13 +0000 (21:25 +0000)
Since https://golang.org/cl/38533, this validation is performed in
driverArgs.

Change-Id: I13a3ca46a1aa3197370de1095fb46ab83ea4628c
Reviewed-on: https://go-review.googlesource.com/91115
Reviewed-by: Daniel Theophanes <kardianos@gmail.com>
Run-TryBot: Daniel Theophanes <kardianos@gmail.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>

src/database/sql/sql.go

index a6fa153d94795486a59e5a957ad7bb285c2f3d2d..5d62eb2b53c53e6e27d505291659d57741048615 100644 (file)
@@ -2262,13 +2262,6 @@ func resultFromStatement(ctx context.Context, ci driver.Conn, ds *driverStmt, ar
                return nil, err
        }
 
-       // -1 means the driver doesn't know how to count the number of
-       // placeholders, so we won't sanity check input here and instead let the
-       // driver deal with errors.
-       if want := ds.si.NumInput(); want >= 0 && want != len(dargs) {
-               return nil, fmt.Errorf("sql: statement expects %d inputs; got %d", want, len(dargs))
-       }
-
        resi, err := ctxDriverStmtExec(ctx, ds.si, dargs)
        if err != nil {
                return nil, err
@@ -2440,13 +2433,6 @@ func rowsiFromStatement(ctx context.Context, ci driver.Conn, ds *driverStmt, arg
                return nil, err
        }
 
-       // -1 means the driver doesn't know how to count the number of
-       // placeholders, so we won't sanity check input here and instead let the
-       // driver deal with errors.
-       if want := ds.si.NumInput(); want >= 0 && want != len(dargs) {
-               return nil, fmt.Errorf("sql: statement expects %d inputs; got %d", want, len(dargs))
-       }
-
        rowsi, err := ctxDriverStmtQuery(ctx, ds.si, dargs)
        if err != nil {
                return nil, err