net: wrong sample code in package document

[gopherbot]

by xiezhenye:

sample of net.Conn.Accept：

ln, err := net.Listen("tcp", ":8080")
if err != nil {
    // handle error
}
for {
    conn, err := ln.Accept()
    if err != nil {
        // handle error
        continue
    }
    go handleConnection(conn)
}

When err that Accept returns is not nil, it must not continue the loop.
recoverable err like syscall.EAGAIN, syscall.ECONNABORTED has been handler in Accept,
and all the err that returned is not recoverable. continue the loop will cause dead loop.

the sample should be:

ln, err := net.Listen("tcp", ":8080")
if err != nil {
    // handle error
}
for {
    conn, err := ln.Accept()
    if err != nil {
        // handle error
        break
    }
    go handleConnection(conn)
}

[mikioh]

Comment 1:

Mr. Fitzpatrick wrote it, so leave it to him.

Owner changed to @bradfitz.

[bradfitz]

Comment 2:

https://golang.org/cl/146470043

Labels changed: added release-go1.4, repo-main, documentation.

Status changed to Started.

[gopherbot]

Comment 3:

CL https://golang.org/cl/146470043 mentions this issue.

[bradfitz]

Comment 4:

This issue was closed by revision dca4605.

Status changed to Fixed.