Refactor by removing unnecessary else block (#1559)

valyala · May 11, 2023 · eaab8ff · eaab8ff
1 parent 1a46e57
commit eaab8ff
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 12 deletions.
diff --git a/args.go b/args.go
@@ -598,9 +598,8 @@ func decodeArgAppendNoPlus(dst, src []byte) []byte {
 	if idx < 0 {
 		// fast path: src doesn't contain encoded chars
 		return append(dst, src...)
-	} else {
-		dst = append(dst, src[:idx]...)
 	}
+	dst = append(dst, src[:idx]...)
 
 	// slow path
 	for i := idx; i < len(src); i++ {

diff --git a/streaming.go b/streaming.go
@@ -72,16 +72,15 @@ func (rs *requestStream) Read(p []byte) (int, error) {
 			return n, io.EOF
 		}
 		return n, err
-	} else {
-		left := rs.header.ContentLength() - rs.totalBytesRead
-		if len(p) > left {
-			p = p[:left]
-		}
-		n, err = rs.reader.Read(p)
-		rs.totalBytesRead += n
-		if err != nil {
-			return n, err
-		}
+	}
+	left := rs.header.ContentLength() - rs.totalBytesRead
+	if len(p) > left {
+		p = p[:left]
+	}
+	n, err = rs.reader.Read(p)
+	rs.totalBytesRead += n
+	if err != nil {
+		return n, err
 	}
 
 	if rs.totalBytesRead == rs.header.ContentLength() {