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

Delta Between Two Patch Sets: src/pkg/os/os_test.go

Issue 4628045: code review 4628045: os: Change Waitmsg.String() to operate on pointer.
Left Patch Set: diff -r dc6d3cf9279d https://go.googlecode.com/hg/ Created 13 years, 9 months ago
Right Patch Set: diff -r 37a78ff6db53 https://go.googlecode.com/hg/ Created 13 years, 9 months ago
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments. Please Sign in to add in-line comments.
Jump to:
Right: Side by side diff | Download
« no previous file with change/comment | « src/pkg/os/exec_posix.go ('k') | no next file » | no next file with change/comment »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
LEFTRIGHT
(no file at all)
1 // Copyright 2009 The Go Authors. All rights reserved. 1 // Copyright 2009 The Go Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style 2 // Use of this source code is governed by a BSD-style
3 // license that can be found in the LICENSE file. 3 // license that can be found in the LICENSE file.
4 4
5 package os_test 5 package os_test
6 6
7 import ( 7 import (
8 "bytes" 8 "bytes"
9 "fmt" 9 "fmt"
10 "io" 10 "io"
(...skipping 1024 matching lines...) Expand 10 before | Expand all | Expand 10 after
1035 if err != nil { 1035 if err != nil {
1036 t.Fatal("stat failed:", err) 1036 t.Fatal("stat failed:", err)
1037 } 1037 }
1038 1038
1039 // Stat of path+"/" should succeed too. 1039 // Stat of path+"/" should succeed too.
1040 _, err = Stat(path + "/") 1040 _, err = Stat(path + "/")
1041 if err != nil { 1041 if err != nil {
1042 t.Fatal("stat failed:", err) 1042 t.Fatal("stat failed:", err)
1043 } 1043 }
1044 } 1044 }
1045
1046 func TestNilWaitmsgString(t *testing.T) {
1047 var w *Waitmsg
1048 s := w.String()
1049 if s != "<nil>" {
1050 t.Errorf("(*Waitmsg)(nil).String() = %q, want %q", s, "<nil>")
1051 }
1052 }
LEFTRIGHT

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