Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(129)

Unified Diff: src/pkg/net/http/transport_test.go

Issue 96850043: code review 96850043: src: fix issues found by go vet std (Closed)
Patch Set: diff -r e141dda580af https://code.google.com/p/go Created 10 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Please Sign in to add in-line comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « src/pkg/net/http/serve_test.go ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/pkg/net/http/transport_test.go
===================================================================
--- a/src/pkg/net/http/transport_test.go
+++ b/src/pkg/net/http/transport_test.go
@@ -1412,7 +1412,6 @@
case <-gotres:
case <-time.After(5 * time.Second):
panic("hang. events are: " + logbuf.String())
- t.Fatal("timeout; cancel didn't work?")
}
got := logbuf.String()
@@ -1869,10 +1868,10 @@
return
}
if !ne.Timeout() {
- t.Error("expected timeout error; got %v", err)
+ t.Errorf("expected timeout error; got %v", err)
}
if !strings.Contains(err.Error(), "handshake timeout") {
- t.Error("expected 'handshake timeout' in error; got %v", err)
+ t.Errorf("expected 'handshake timeout' in error; got %v", err)
}
}()
select {
« no previous file with comments | « src/pkg/net/http/serve_test.go ('k') | no next file » | no next file with comments »

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b