From d24cf6d53e854d0f7cda6912575c7054637ed53c Mon Sep 17 00:00:00 2001 From: Dmitri Shuralyov Date: Mon, 21 Oct 2019 15:04:54 -0400 Subject: [PATCH] database/sql: remove forced log import from test This var _ = log.Printf line was added 8 years ago, in CL 4973055, which created the database/sql package and its tests. There was no goimports back then, so this was likely added to make it easier to use log package during development of tests. It's no longer needed, so remove it. It can always be conveniently re-added via goimports whenever needed. Change-Id: I7c32ae3e593c194d970920084139dfa5a42386dc Reviewed-on: https://go-review.googlesource.com/c/go/+/202481 Run-TryBot: Dmitri Shuralyov TryBot-Result: Gobot Gobot Reviewed-by: Brad Fitzpatrick --- src/database/sql/fakedb_test.go | 3 --- 1 file changed, 3 deletions(-) diff --git a/src/database/sql/fakedb_test.go b/src/database/sql/fakedb_test.go index c0371f3e78..a0028be0e5 100644 --- a/src/database/sql/fakedb_test.go +++ b/src/database/sql/fakedb_test.go @@ -10,7 +10,6 @@ import ( "errors" "fmt" "io" - "log" "reflect" "sort" "strconv" @@ -20,8 +19,6 @@ import ( "time" ) -var _ = log.Printf - // fakeDriver is a fake database that implements Go's driver.Driver // interface, just for testing. // -- 2.50.0