database/sql: refactor unused variables to use blank identifiers

Hi
I've replaced unused variables in database/sql with blank identifiers to improve code readability.

This change has no impact on the functionality of the code, but makes it easier to read and understand.

Change-Id: I701e93fd9bf86725f411085bf2e8f8e6b235af14
GitHub-Last-Rev: eb40bd5e2ef9e28871fae2e97b4433e8d6e2c652
GitHub-Pull-Request: golang/go#58986
Reviewed-on: https://go-review.googlesource.com/c/go/+/475675
Commit-Queue: Ian Lance Taylor <iant@google.com>
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
Reviewed-by: Damien Neil <dneil@google.com>
Auto-Submit: Ian Lance Taylor <iant@google.com>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Reviewed-by: Ian Lance Taylor <iant@google.com>

diff --git a/src/database/sql/convert.go b/src/database/sql/convert.go
index b291c1557c7ab616a800564ecdc8e5ec92160402..dac3f246ae63e56228f16e0158e357aadab297c3 100644 (file)
--- a/src/database/sql/convert.go
+++ b/src/database/sql/convert.go
@@ -127,7 +127,7 @@ func driverArgsConnLocked(ci driver.Conn, ds *driverStmt, args []any) ([]driver.
        // to the column converter.
        nvc, ok := si.(driver.NamedValueChecker)
        if !ok {
-               nvc, ok = ci.(driver.NamedValueChecker)
+               nvc, _ = ci.(driver.NamedValueChecker)
        }
        cci, ok := si.(driver.ColumnConverter)
        if ok {
@@ -136,7 +136,7 @@ func driverArgsConnLocked(ci driver.Conn, ds *driverStmt, args []any) ([]driver.
 
        // Loop through all the arguments, checking each one.
        // If no error is returned simply increment the index
-       // and continue. However if driver.ErrRemoveArgument
+       // and continue. However, if driver.ErrRemoveArgument
        // is returned the argument is not included in the query
        // argument list.
        var err error