From e565720e4920fe14eaeac55c8a271444d5499e44 Mon Sep 17 00:00:00 2001 From: Mohammad Abbasi Date: Sun, 12 Mar 2023 00:23:14 +0000 Subject: [PATCH] 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 Reviewed-by: Brad Fitzpatrick Reviewed-by: Damien Neil Auto-Submit: Ian Lance Taylor LUCI-TryBot-Result: Go LUCI Reviewed-by: Ian Lance Taylor --- src/database/sql/convert.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/database/sql/convert.go b/src/database/sql/convert.go index b291c1557c..dac3f246ae 100644 --- 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 -- 2.48.1