Fix go vet warnings.

Mostly unreachable code, one format string issue.

Signed-off-by: Paul Nasrat <pnasrat@google.com> (github: pnasrat)

Author: Paul Nasrat

copy.go

@@ -34,5 +34,4 @@ func Copy(dst Sender, src Receiver) (int, error) {
 		}
 		n++
 	}
-	return n, nil
 }

data/message.go

@@ -44,7 +44,6 @@ func (m Message) Set(k string, v ...string) Message {
 	data, err := Decode(string(m))
 	if err != nil {
 		panic(err)
-		return m
 	}
 	data[k] = v
 	return Message(Encode(data))
@@ -54,7 +53,6 @@ func (m Message) Del(k string) Message {
 	data, err := Decode(string(m))
 	if err != nil {
 		panic(err)
-		return m
 	}
 	delete(data, k)
 	return Message(Encode(data))

unix/libchan.go

@@ -113,8 +113,6 @@ func ReceiveConn(src Receiver) ([]byte, *UnixConn, error) {
 		}
 		return data, conn, nil
 	}
-	panic("impossibru!")
-	return nil, nil, nil
 }
 
 func Copy(dst Sender, src Receiver) (int, error) {
@@ -134,8 +132,6 @@ func Copy(dst Sender, src Receiver) (int, error) {
 		}
 		n++
 	}
-	panic("impossibru!")
-	return n, nil
 }
 
 // MsgDesc returns a human readable description of a libchan message, usually

utils/stack_test.go

@@ -110,7 +110,7 @@ func TestStackAddBad(t *testing.T) {
 		t.Fatal(err)
 	}
 	if s.Len() != 1 {
-		t.Fatalf("%#v")
+		t.Fatalf("%#v", s)
 	}
 	if len(buf) != 1 {
 		t.Fatalf("%#v", buf)