Ineffectual assignments

cmd/ck-client/ck-client.go

@@ -138,7 +138,7 @@ start:
 				stream.Close()
 				break
 			}
-			i, err = localConn.WriteToUDP(buf[:i], otherEnd.Load().(*net.UDPAddr))
+			_, err = localConn.WriteToUDP(buf[:i], otherEnd.Load().(*net.UDPAddr))
 			if err != nil {
 				log.Print(err)
 				localConn.Close()
@@ -164,7 +164,7 @@ start:
 			break
 		}
 		otherEnd.Store(oe)
-		i, err = stream.Write(buf[:i])
+		_, err = stream.Write(buf[:i])
 		if err != nil {
 			localConn.Close()
 			stream.Close()

internal/multiplex/session_test.go

@@ -172,7 +172,7 @@ func TestRecvDataFromRemote_Closing_InOrder(t *testing.T) {
 	if err != nil {
 		t.Fatalf("receiving normal frame for stream 1: %v", err)
 	}
-	s1I, ok := sesh.streams.Load(f1.StreamID)
+	_, ok := sesh.streams.Load(f1.StreamID)
 	if !ok {
 		t.Fatal("failed to fetch stream 1 after receiving it")
 	}

internal/util/util.go

@@ -68,7 +68,7 @@ func ReadTLS(conn net.Conn, buffer []byte) (n int, err error) {
 	// a single message can also be segmented due to MTU of the IP layer.
 	// This function guareentees a single TLS message to be read and everything
 	// else is left in the buffer.
-	i, err := io.ReadFull(conn, buffer[:5])
+	_, err = io.ReadFull(conn, buffer[:5])
 	if err != nil {
 		return
 	}
@@ -87,7 +87,7 @@ func ReadTLS(conn net.Conn, buffer []byte) (n int, err error) {
 		// if left < buffer size (i.e. multiple messages came together),
 		// only the message we want is read
 
-		i, err = conn.Read(buffer[readPtr : readPtr+left])
+		i, err := conn.Read(buffer[readPtr : readPtr+left])
 		if err != nil {
 			return
 		}
@@ -129,7 +129,7 @@ func Pipe(dst net.Conn, src net.Conn, srcReadTimeout time.Duration) {
 			src.Close()
 			return
 		}
-		i, err = dst.Write(buf[:i])
+		_, err = dst.Write(buf[:i])
 		if err != nil {
 			dst.Close()
 			src.Close()