]> Cypherpunks repositories - gostls13.git/commitdiff
all: remove now-unnecessary unreachable panics
authorBrad Fitzpatrick <bradfitz@golang.org>
Mon, 11 Mar 2013 21:16:55 +0000 (14:16 -0700)
committerBrad Fitzpatrick <bradfitz@golang.org>
Mon, 11 Mar 2013 21:16:55 +0000 (14:16 -0700)
Take advantage of the new terminating statement rule.

R=golang-dev, r, gri
CC=golang-dev
https://golang.org/cl/7712044

32 files changed:
src/cmd/api/goapi.go
src/cmd/go/build.go
src/pkg/bufio/bufio.go
src/pkg/bufio/scan.go
src/pkg/compress/bzip2/huffman.go
src/pkg/compress/flate/deflate_test.go
src/pkg/compress/flate/inflate.go
src/pkg/compress/flate/token.go
src/pkg/compress/lzw/reader.go
src/pkg/crypto/dsa/dsa.go
src/pkg/crypto/x509/pkcs8.go
src/pkg/crypto/x509/x509.go
src/pkg/encoding/ascii85/ascii85.go
src/pkg/encoding/binary/varint.go
src/pkg/encoding/csv/reader.go
src/pkg/encoding/json/decode.go
src/pkg/encoding/xml/read.go
src/pkg/go/parser/error_test.go
src/pkg/html/template/transition.go
src/pkg/image/gif/reader.go
src/pkg/io/ioutil/ioutil.go
src/pkg/math/rand/exp.go
src/pkg/math/rand/normal.go
src/pkg/mime/multipart/multipart.go
src/pkg/net/dial.go
src/pkg/net/http/client_test.go
src/pkg/net/http/fcgi/child.go
src/pkg/net/http/server.go
src/pkg/net/http/transport.go
src/pkg/net/rpc/server_test.go
src/pkg/net/textproto/reader.go
src/pkg/os/file_unix.go

index adf3ddf6b39d3102a223f4b7e306a355b7f07951..d5e66428b779f2f09b8ced895f52e18bc66c3043 100644 (file)
@@ -723,7 +723,6 @@ func (w *Walker) varValueType(vi interface{}) (string, error) {
        default:
                return "", fmt.Errorf("unknown const value type %T", vi)
        }
-       panic("unreachable")
 }
 
 // resolveName finds a top-level node named name and returns the node
index 83aeedaca9dd91119a483c2f177beb79b4d3e60c..1967c8457feb77614d0bc12ec3b512e409037f67 100644 (file)
@@ -1291,7 +1291,6 @@ func (b *builder) runOut(dir string, desc string, cmdargs ...interface{}) ([]byt
 
                return buf.Bytes(), err
        }
-       panic("unreachable")
 }
 
 // mkdir makes the named directory.
index ee69c2d31e4f55bab7b7498f3d370b3c85041ac9..df3501f2ca1776dfae856aa2bfa8c4e3002aa803 100644 (file)
@@ -274,7 +274,6 @@ func (b *Reader) ReadSlice(delim byte) (line []byte, err error) {
                        return b.buf, ErrBufferFull
                }
        }
-       panic("not reached")
 }
 
 // ReadLine is a low-level line-reading primitive. Most callers should use
index 268ce6d1d31d10e3d5c2cbb756daa83f433e287c..b3c652911cfc91cc365190cef32ba25d4fe9144d 100644 (file)
@@ -169,7 +169,6 @@ func (s *Scanner) Scan() bool {
                }
                s.end += n
        }
-       panic("not reached")
 }
 
 // advance consumes n bytes of the buffer. It reports whether the advance was legal.
index 078c1cb895411bd601cedd718e3ba2a466ea6e94..f755019bb50eedc8804303b7773bc9a071c3daa7 100644 (file)
@@ -54,8 +54,6 @@ func (t huffmanTree) Decode(br *bitReader) (v uint16) {
                        nodeIndex = node.right
                }
        }
-
-       panic("unreachable")
 }
 
 // newHuffmanTree builds a Huffman tree from a slice containing the code
index 8f4e196b4274ee3fe459086169fb26dafd166e33..8c4a6d6b36fab8a6526d335d15836c4f17ae00b0 100644 (file)
@@ -158,7 +158,6 @@ func (b *syncBuffer) Read(p []byte) (n int, err error) {
                }
                <-b.ready
        }
-       panic("unreachable")
 }
 
 func (b *syncBuffer) signal() {
index a8d64601925377cc8db8bf70ad110fbae293b0c0..7fe1749631092b81545e9fd45297b1011018c7ca 100644 (file)
@@ -263,7 +263,6 @@ func (f *decompressor) Read(b []byte) (int, error) {
                }
                f.step(f)
        }
-       panic("unreachable")
 }
 
 func (f *decompressor) Close() error {
@@ -495,7 +494,6 @@ func (f *decompressor) huffmanBlock() {
                        return
                }
        }
-       panic("unreached")
 }
 
 // copyHist copies f.copyLen bytes from f.hist (f.copyDist bytes ago) to itself.
index 38aea5fa65056888b427383e819a2fd3e5cab9e6..4d491768717f8ccd945130ad8ff49618db3d8fa9 100644 (file)
@@ -99,5 +99,4 @@ func offsetCode(off uint32) uint32 {
        default:
                return offsetCodes[off>>14] + 28
        }
-       panic("unreachable")
 }
index 0ed742c897774af7515f3fad0b41946b0eca42c1..cf8ca7c788a6fe987f2b917b3b53d9bf61b688d4 100644 (file)
@@ -121,7 +121,6 @@ func (d *decoder) Read(b []byte) (int, error) {
                }
                d.decode()
        }
-       panic("unreachable")
 }
 
 // decode decompresses bytes from r and leaves them in d.toRead.
index 05766a2f13684411e927eef28dd48ee28b11fd80..5a2a65744ed301e79bbf83653cf9601b6217966f 100644 (file)
@@ -144,8 +144,6 @@ GeneratePrimes:
                params.G = g
                return
        }
-
-       panic("unreachable")
 }
 
 // GenerateKey generates a public&private key pair. The Parameters of the
index 30caacb3c54df35c22cabdb26651354aa5a7d1a2..8e1585e15cc840571c634e9f14c0b300d9cf50a9 100644 (file)
@@ -51,6 +51,4 @@ func ParsePKCS8PrivateKey(der []byte) (key interface{}, err error) {
        default:
                return nil, fmt.Errorf("crypto/x509: PKCS#8 wrapping contained private key with unknown algorithm: %v", privKey.Algo.Algorithm)
        }
-
-       panic("unreachable")
 }
index b802bf4ebf20bc78681d36a462fc6130a359b8d0..4dfea2c9499312d1b408eed75ef77bcdc5d36d92 100644 (file)
@@ -729,7 +729,6 @@ func parsePublicKey(algo PublicKeyAlgorithm, keyData *publicKeyInfo) (interface{
        default:
                return nil, nil
        }
-       panic("unreachable")
 }
 
 func parseCertificate(in *certificate) (*Certificate, error) {
index 705022792a863bc417179c261467db2251c48da4..e2afc587140a3fedfe00cb1966105fd38668e0db 100644 (file)
@@ -296,5 +296,4 @@ func (d *decoder) Read(p []byte) (n int, err error) {
                nn, d.readErr = d.r.Read(d.buf[d.nbuf:])
                d.nbuf += nn
        }
-       panic("unreachable")
 }
index 7035529f2791a104b058d8a8c0124a7d6dd19785..3a2dfa3c74e2139199fa3922521d8ca783c7594c 100644 (file)
@@ -120,7 +120,6 @@ func ReadUvarint(r io.ByteReader) (uint64, error) {
                x |= uint64(b&0x7f) << s
                s += 7
        }
-       panic("unreachable")
 }
 
 // ReadVarint reads an encoded signed integer from r and returns it as an int64.
index db4d988526f0623165d380c398a85888d528e269..336dd6540d772383dace1c15f224149dc71b3afc 100644 (file)
@@ -171,7 +171,6 @@ func (r *Reader) ReadAll() (records [][]string, err error) {
                }
                records = append(records, record)
        }
-       panic("unreachable")
 }
 
 // readRune reads one rune from r, folding \r\n to \n and keeping track
index f2ec9cb67240cd3ef4472510062b5bc3e2ee4df4..d34e10f838456862927d9c48d6161787b5ceacf3 100644 (file)
@@ -739,6 +739,7 @@ func (d *decodeState) valueInterface() interface{} {
        switch d.scanWhile(scanSkipSpace) {
        default:
                d.error(errPhase)
+               panic("unreachable")
        case scanBeginArray:
                return d.arrayInterface()
        case scanBeginObject:
@@ -746,7 +747,6 @@ func (d *decodeState) valueInterface() interface{} {
        case scanBeginLiteral:
                return d.literalInterface()
        }
-       panic("unreachable")
 }
 
 // arrayInterface is like array but returns []interface{}.
index 344ab514e378bee139b29ba3f90fbc9cfac0bb40..1581705efb53f63ae6ee9c01fa20f0030a849604 100644 (file)
@@ -493,7 +493,6 @@ Loop:
                        return true, nil
                }
        }
-       panic("unreachable")
 }
 
 // Skip reads tokens until it has consumed the end element
@@ -517,5 +516,4 @@ func (d *Decoder) Skip() error {
                        return nil
                }
        }
-       panic("unreachable")
 }
index b59fda11a3be545f0a519b857d018c0fadddb2d5..d4d4f909d3ffd909c5dfa36bae9f6c3e8c745d00 100644 (file)
@@ -89,8 +89,6 @@ func expectedErrors(t *testing.T, filename string, src []byte) map[token.Pos]str
                        prev = pos
                }
        }
-
-       panic("unreachable")
 }
 
 // compareErrors compares the map of expected error messages with the list
index 96a4f6678bc746483636afeee048a73e70fcd34f..2ea2089c6b034b698320999e749a07fada2d0aaa 100644 (file)
@@ -71,7 +71,6 @@ func tText(c context, s []byte) (context, int) {
                }
                k = j
        }
-       panic("unreachable")
 }
 
 var elementContentType = [...]state{
index 8b36948d6993b9bbd84fcfd8ce16bbc96f5196bc..ed493eac2f95928c95f24225bf6c084265ac65f5 100644 (file)
@@ -304,7 +304,6 @@ func (d *decoder) readExtension() error {
                        return err
                }
        }
-       panic("unreachable")
 }
 
 func (d *decoder) readGraphicControl() error {
index 0eb146c0ab75598d082aeb4d247d0bd48629874a..6b395c69bdc06669ec4a4418e2667ed08fbc1b25 100644 (file)
@@ -144,7 +144,6 @@ func (devNull) ReadFrom(r io.Reader) (n int64, err error) {
                        return
                }
        }
-       panic("unreachable")
 }
 
 // Discard is an io.Writer on which all Write calls succeed
index 85da495219d5126851690656638207c696c06885..4bc110f91380c70a18237519100343e91b1c1e53 100644 (file)
@@ -43,7 +43,6 @@ func (r *Rand) ExpFloat64() float64 {
                        return x
                }
        }
-       panic("unreachable")
 }
 
 var ke = [256]uint32{
index 9ab46db9f53ce4f1856e5efbd58f5cab2d679ec4..ba4ea54cace6801417858053441bcf97b0845cab 100644 (file)
@@ -63,7 +63,6 @@ func (r *Rand) NormFloat64() float64 {
                        return x
                }
        }
-       panic("unreachable")
 }
 
 var kn = [128]uint32{
index 77e969b41b0ea7b924308572cf6bc3e5d941bee1..0592f6977930a30ea5760bdc63518c1ce3bee4e2 100644 (file)
@@ -265,7 +265,6 @@ func (r *Reader) NextPart() (*Part, error) {
 
                return nil, fmt.Errorf("multipart: unexpected line in Next(): %q", line)
        }
-       panic("unreachable")
 }
 
 // isFinalBoundary returns whether line is the final boundary line
index 2e15c06cc949a0435fd34bbd828b951f83763f88..c0e4a2236e05fa6570ad7888896bae98c169ca02 100644 (file)
@@ -280,7 +280,6 @@ func dialTimeoutRace(net, addr string, timeout time.Duration) (Conn, error) {
        case p := <-ch:
                return p.Conn, p.error
        }
-       panic("unreachable")
 }
 
 type stringAddr struct {
index 88649bb167311930a28b6ff0f8d75a6bcc5c32e8..4d40dc972d02261ad371125d1fb675acb2ca26e6 100644 (file)
@@ -51,7 +51,6 @@ func pedanticReadAll(r io.Reader) (b []byte, err error) {
                        return b, err
                }
        }
-       panic("unreachable")
 }
 
 func TestClient(t *testing.T) {
index c8b9a33c87bedbfc2757db6ffc1a1411b38fa976..e647f9391e458d61e95da93d6c6f3779ccc62387 100644 (file)
@@ -267,5 +267,4 @@ func Serve(l net.Listener, handler http.Handler) error {
                c := newChild(rw, handler)
                go c.serve()
        }
-       panic("unreachable")
 }
index b6ab782286f3c5c6e2b86e7e55d015c28218a1de..902176716322a2189a9a636e27fb0cd23e68f78e 100644 (file)
@@ -1337,7 +1337,6 @@ func (srv *Server) Serve(l net.Listener) error {
                }
                go c.serve()
        }
-       panic("not reached")
 }
 
 // ListenAndServe listens on the TCP network address addr
index f3aaa79ccedf8c6d784478e2da4eae221c7f5f82..f1c6fb2dcb75ebc0a38f860a0664e6afc56f838a 100644 (file)
@@ -349,7 +349,6 @@ func (t *Transport) getIdleConn(cm *connectMethod) (pconn *persistConn) {
                        return
                }
        }
-       panic("unreachable")
 }
 
 func (t *Transport) setReqConn(r *Request, pc *persistConn) {
index 5b2f9f2ded82db42fbd8205dcd04077a2bbe8601..eb17210abc91157e209cfbee443de183e7148cdc 100644 (file)
@@ -399,12 +399,10 @@ func (WriteFailCodec) WriteRequest(*Request, interface{}) error {
 
 func (WriteFailCodec) ReadResponseHeader(*Response) error {
        select {}
-       panic("unreachable")
 }
 
 func (WriteFailCodec) ReadResponseBody(interface{}) error {
        select {}
-       panic("unreachable")
 }
 
 func (WriteFailCodec) Close() error {
index b61bea862191a90b12550974dcae76949ed62dc4..35e27acb5a60a88931b4a2408c3e766e11ff131c 100644 (file)
@@ -489,7 +489,6 @@ func (r *Reader) ReadMIMEHeader() (MIMEHeader, error) {
                        return m, err
                }
        }
-       panic("unreachable")
 }
 
 // CanonicalMIMEHeaderKey returns the canonical format of the
index 4f59c94cb9495d47d101fc4b941be7d0087d6919..898e7634a7ef3d3e615ed2a1a024e6f406896a41 100644 (file)
@@ -198,7 +198,6 @@ func (f *File) write(b []byte) (n int, err error) {
 
                return n, err
        }
-       panic("not reached")
 }
 
 // pwrite writes len(b) bytes to the File starting at byte offset off.