Skip to content

Commit 6aaef04

Browse files
committed
misc cleanup: gofmt + &x -> x[0:] conversion
R=gri CC=golang-dev https://golang.org/cl/1620042
1 parent f29fb25 commit 6aaef04

File tree

10 files changed

+149
-141
lines changed

10 files changed

+149
-141
lines changed

src/pkg/debug/macho/file.go

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -168,11 +168,11 @@ func NewFile(r io.ReaderAt) (*File, os.Error) {
168168
// Read and decode Mach magic to determine byte order, size.
169169
// Magic32 and Magic64 differ only in the bottom bit.
170170
var ident [4]byte
171-
if _, err := r.ReadAt(&ident, 0); err != nil {
171+
if _, err := r.ReadAt(ident[0:], 0); err != nil {
172172
return nil, err
173173
}
174-
be := binary.BigEndian.Uint32(&ident)
175-
le := binary.LittleEndian.Uint32(&ident)
174+
be := binary.BigEndian.Uint32(ident[0:])
175+
le := binary.LittleEndian.Uint32(ident[0:])
176176
switch Magic32 &^ 1 {
177177
case be &^ 1:
178178
f.ByteOrder = binary.BigEndian
@@ -227,7 +227,7 @@ func NewFile(r io.ReaderAt) (*File, os.Error) {
227227
s.LoadBytes = cmddat
228228
s.Cmd = cmd
229229
s.Len = siz
230-
s.Name = cstring(&seg32.Name)
230+
s.Name = cstring(seg32.Name[0:])
231231
s.Addr = uint64(seg32.Addr)
232232
s.Memsz = uint64(seg32.Memsz)
233233
s.Offset = uint64(seg32.Offset)
@@ -243,8 +243,8 @@ func NewFile(r io.ReaderAt) (*File, os.Error) {
243243
return nil, err
244244
}
245245
sh := new(Section)
246-
sh.Name = cstring(&sh32.Name)
247-
sh.Seg = cstring(&sh32.Seg)
246+
sh.Name = cstring(sh32.Name[0:])
247+
sh.Seg = cstring(sh32.Seg[0:])
248248
sh.Addr = uint64(sh32.Addr)
249249
sh.Size = uint64(sh32.Size)
250250
sh.Offset = sh32.Offset
@@ -265,7 +265,7 @@ func NewFile(r io.ReaderAt) (*File, os.Error) {
265265
s.LoadBytes = cmddat
266266
s.Cmd = cmd
267267
s.Len = siz
268-
s.Name = cstring(&seg64.Name)
268+
s.Name = cstring(seg64.Name[0:])
269269
s.Addr = seg64.Addr
270270
s.Memsz = seg64.Memsz
271271
s.Offset = seg64.Offset
@@ -281,8 +281,8 @@ func NewFile(r io.ReaderAt) (*File, os.Error) {
281281
return nil, err
282282
}
283283
sh := new(Section)
284-
sh.Name = cstring(&sh64.Name)
285-
sh.Seg = cstring(&sh64.Seg)
284+
sh.Name = cstring(sh64.Name[0:])
285+
sh.Seg = cstring(sh64.Seg[0:])
286286
sh.Addr = sh64.Addr
287287
sh.Size = sh64.Size
288288
sh.Offset = sh64.Offset

src/pkg/debug/proc/regs_linux_386.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ type _386Regs struct {
1515
setter func(*syscall.PtraceRegs) os.Error
1616
}
1717

18-
var names = [...]string{
18+
var names = []string{
1919
"eax",
2020
"ebx",
2121
"ecx",
@@ -55,7 +55,7 @@ func (r *_386Regs) SetSP(val Word) os.Error {
5555
return r.setter(&r.PtraceRegs)
5656
}
5757

58-
func (r *_386Regs) Names() []string { return &names }
58+
func (r *_386Regs) Names() []string { return names }
5959

6060
func (r *_386Regs) Get(i int) Word {
6161
switch i {

src/pkg/http/triv.go

Lines changed: 11 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -6,13 +6,12 @@ package main
66

77
import (
88
"bytes"
9-
"bufio"
109
"expvar"
1110
"flag"
1211
"fmt"
12+
"http"
1313
"io"
1414
"log"
15-
"net"
1615
"os"
1716
"strconv"
1817
)
@@ -67,7 +66,7 @@ func FileServer(c *http.Conn, req *http.Request) {
6766
fmt.Fprintf(c, "open %s: %v\n", path, err)
6867
return
6968
}
70-
n, err1 := io.Copy(c, f)
69+
n, _ := io.Copy(c, f)
7170
fmt.Fprintf(c, "[%d bytes]\n", n)
7271
f.Close()
7372
}
@@ -89,7 +88,7 @@ func FlagServer(c *http.Conn, req *http.Request) {
8988

9089
// simple argument server
9190
func ArgServer(c *http.Conn, req *http.Request) {
92-
for i, s := range os.Args {
91+
for _, s := range os.Args {
9392
fmt.Fprint(c, s, " ")
9493
}
9594
}
@@ -138,6 +137,13 @@ func DateServer(c *http.Conn, req *http.Request) {
138137
}
139138
}
140139

140+
func Logger(c *http.Conn, req *http.Request) {
141+
log.Stdout(req.URL.Raw)
142+
c.WriteHeader(404)
143+
c.Write([]byte("oops"))
144+
}
145+
146+
141147
func main() {
142148
flag.Parse()
143149

@@ -146,6 +152,7 @@ func main() {
146152
http.Handle("/counter", ctr)
147153
expvar.Publish("counter", ctr)
148154

155+
http.Handle("/", http.HandlerFunc(Logger))
149156
http.Handle("/go/", http.HandlerFunc(FileServer))
150157
http.Handle("/flags", http.HandlerFunc(FlagServer))
151158
http.Handle("/args", http.HandlerFunc(ArgServer))

src/pkg/json/stream_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -109,7 +109,7 @@ func TestRawMessage(t *testing.T) {
109109
if err != nil {
110110
t.Fatalf("Unmarshal: %v", err)
111111
}
112-
if string(*data.Id) != raw {
112+
if string([]byte(*data.Id)) != raw {
113113
t.Fatalf("Raw mismatch: have %#q want %#q", []byte(*data.Id), raw)
114114
}
115115
b, err := Marshal(&data)

src/pkg/net/iprawsock.go

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -15,9 +15,9 @@ import (
1515
func sockaddrToIP(sa syscall.Sockaddr) Addr {
1616
switch sa := sa.(type) {
1717
case *syscall.SockaddrInet4:
18-
return &IPAddr{&sa.Addr}
18+
return &IPAddr{sa.Addr[0:]}
1919
case *syscall.SockaddrInet6:
20-
return &IPAddr{&sa.Addr}
20+
return &IPAddr{sa.Addr[0:]}
2121
}
2222
return nil
2323
}
@@ -176,14 +176,14 @@ func (c *IPConn) ReadFromIP(b []byte) (n int, addr *IPAddr, err os.Error) {
176176
n, sa, err := c.fd.ReadFrom(b)
177177
switch sa := sa.(type) {
178178
case *syscall.SockaddrInet4:
179-
addr = &IPAddr{&sa.Addr}
179+
addr = &IPAddr{sa.Addr[0:]}
180180
if len(b) >= 4 { // discard ipv4 header
181181
hsize := (int(b[0]) & 0xf) * 4
182182
copy(b, b[hsize:])
183183
n -= hsize
184184
}
185185
case *syscall.SockaddrInet6:
186-
addr = &IPAddr{&sa.Addr}
186+
addr = &IPAddr{sa.Addr[0:]}
187187
}
188188
return
189189
}

src/pkg/net/tcpsock.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,9 +14,9 @@ import (
1414
func sockaddrToTCP(sa syscall.Sockaddr) Addr {
1515
switch sa := sa.(type) {
1616
case *syscall.SockaddrInet4:
17-
return &TCPAddr{&sa.Addr, sa.Port}
17+
return &TCPAddr{sa.Addr[0:], sa.Port}
1818
case *syscall.SockaddrInet6:
19-
return &TCPAddr{&sa.Addr, sa.Port}
19+
return &TCPAddr{sa.Addr[0:], sa.Port}
2020
}
2121
return nil
2222
}

src/pkg/net/udpsock.go

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -14,9 +14,9 @@ import (
1414
func sockaddrToUDP(sa syscall.Sockaddr) Addr {
1515
switch sa := sa.(type) {
1616
case *syscall.SockaddrInet4:
17-
return &UDPAddr{&sa.Addr, sa.Port}
17+
return &UDPAddr{sa.Addr[0:], sa.Port}
1818
case *syscall.SockaddrInet6:
19-
return &UDPAddr{&sa.Addr, sa.Port}
19+
return &UDPAddr{sa.Addr[0:], sa.Port}
2020
}
2121
return nil
2222
}
@@ -176,9 +176,9 @@ func (c *UDPConn) ReadFromUDP(b []byte) (n int, addr *UDPAddr, err os.Error) {
176176
n, sa, err := c.fd.ReadFrom(b)
177177
switch sa := sa.(type) {
178178
case *syscall.SockaddrInet4:
179-
addr = &UDPAddr{&sa.Addr, sa.Port}
179+
addr = &UDPAddr{sa.Addr[0:], sa.Port}
180180
case *syscall.SockaddrInet6:
181-
addr = &UDPAddr{&sa.Addr, sa.Port}
181+
addr = &UDPAddr{sa.Addr[0:], sa.Port}
182182
}
183183
return
184184
}

0 commit comments

Comments
 (0)