commit eb8f2a832078d0748bcf1eb38357c8119fdbccb7
author Josh Bleecher Snyder <josharian@gmail.com> Fri Nov 11 07:37:32 2016 -0800
committer Josh Bleecher Snyder <josharian@gmail.com> Fri Nov 11 16:01:59 2016 +0000
tree 2045365c040fb435b8b132a306247433159324f1
parent 50fed64dd90ca6a58cfe8fa7c1061aa8666cc76f

all: fix vet nits

Fixes these complaints from vet:

cmd/compile/internal/gc/noder.go:32: cmd/compile/internal/syntax.Error composite literal uses unkeyed fields
cmd/compile/internal/gc/noder.go:1035: cmd/compile/internal/syntax.Error composite literal uses unkeyed fields
cmd/compile/internal/gc/noder.go:1051: cmd/compile/internal/syntax.Error composite literal uses unkeyed fields
cmd/compile/internal/syntax/parser_test.go:182: possible formatting directive in Error call
net/http/client_test.go:1334: possible formatting directive in Fatal call

Change-Id: I5f90ec30f3c106c7e66c92e2b6f8d3b4874fec66
Reviewed-on: https://go-review.googlesource.com/33133
Run-TryBot: Josh Bleecher Snyder <josharian@gmail.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>

src/cmd/compile/internal/gc/noder.go

diff --git a/src/cmd/compile/internal/gc/noder.go b/src/cmd/compile/internal/gc/noder.go
index 35cbeb5..f9de48a 100644
--- a/src/cmd/compile/internal/gc/noder.go
+++ b/src/cmd/compile/internal/gc/noder.go
@@ -29,7 +29,7 @@
 
 	if !imported_unsafe {
 		for _, x := range p.linknames {
-			p.error(syntax.Error{0, x, "//go:linkname only allowed in Go files that import \"unsafe\""})
+			p.error(syntax.Error{Line: x, Msg: "//go:linkname only allowed in Go files that import \"unsafe\""})
 		}
 	}
 
@@ -1032,7 +1032,7 @@
 			break
 		}
 		if n > 1e8 {
-			p.error(syntax.Error{pos, line, "line number out of range"})
+			p.error(syntax.Error{Pos: pos, Line: line, Msg: "line number out of range"})
 			errorexit()
 		}
 		if n <= 0 {
@@ -1048,7 +1048,7 @@
 
 		f := strings.Fields(text)
 		if len(f) != 3 {
-			p.error(syntax.Error{pos, line, "usage: //go:linkname localname linkname"})
+			p.error(syntax.Error{Pos: pos, Line: line, Msg: "usage: //go:linkname localname linkname"})
 			break
 		}
 		lookup(f[1]).Linkname = f[2]

src/cmd/compile/internal/syntax/parser_test.go

diff --git a/src/cmd/compile/internal/syntax/parser_test.go b/src/cmd/compile/internal/syntax/parser_test.go
index dc7f91d..c4b43bf 100644
--- a/src/cmd/compile/internal/syntax/parser_test.go
+++ b/src/cmd/compile/internal/syntax/parser_test.go
@@ -179,6 +179,6 @@
 		t.Error("missing io error")
 	}
 	if err != first {
-		t.Error("got %v; want first error %v", err, first)
+		t.Errorf("got %v; want first error %v", err, first)
 	}
 }

src/net/http/client_test.go

diff --git a/src/net/http/client_test.go b/src/net/http/client_test.go
index b1677c1..d96e267 100644
--- a/src/net/http/client_test.go
+++ b/src/net/http/client_test.go
@@ -1331,7 +1331,7 @@
 	cancel()
 	_, err = io.Copy(ioutil.Discard, res.Body)
 	if err != ExportErrRequestCanceled {
-		t.Fatal("error = %v; want errRequestCanceled")
+		t.Fatalf("error = %v; want errRequestCanceled", err)
 	}
 }