From a764746906ad4dd145b95825a06e67e10b1c385c Mon Sep 17 00:00:00 2001 From: yuki0920 Date: Sun, 30 Jun 2024 07:55:45 +0900 Subject: [PATCH] Fix: use `slices.Contains` according to the TODO comment I used `slices.Contains` according to the TODO comment. ``` // TODO replace by slices.Contains when experimental package will be merged to stdlib // https://pkg.go.dev/golang.org/x/exp/slices#Contains ``` --- stdlib/sql.go | 14 ++------------ 1 file changed, 2 insertions(+), 12 deletions(-) diff --git a/stdlib/sql.go b/stdlib/sql.go index 29cd3fbb..cf76900a 100644 --- a/stdlib/sql.go +++ b/stdlib/sql.go @@ -75,6 +75,7 @@ import ( "math" "math/rand" "reflect" + "slices" "strconv" "strings" "sync" @@ -98,7 +99,7 @@ func init() { // if pgx driver was already registered by different pgx major version then we // skip registration under the default name. - if !contains(sql.Drivers(), "pgx") { + if !slices.Contains(sql.Drivers(), "pgx") { sql.Register("pgx", pgxDriver) } sql.Register("pgx/v5", pgxDriver) @@ -120,17 +121,6 @@ func init() { } } -// TODO replace by slices.Contains when experimental package will be merged to stdlib -// https://pkg.go.dev/golang.org/x/exp/slices#Contains -func contains(list []string, y string) bool { - for _, x := range list { - if x == y { - return true - } - } - return false -} - // OptionOpenDB options for configuring the driver when opening a new db pool. type OptionOpenDB func(*connector)