optimized delAllArgs

currantlabs · Mar 11, 2016 · 3f6e5b6 · 3f6e5b6
1 parent e0eac18
commit 3f6e5b6
Showing 1 changed file with 3 additions and 12 deletions.
diff --git a/args.go b/args.go
@@ -287,23 +287,14 @@ func copyArgs(dst, src []argsKV) []argsKV {
 }
 
 func delAllArgs(args []argsKV, key []byte) []argsKV {
-	for {
-		argsNew := delArg(args, key)
-		if len(argsNew) == len(args) {
-			return args
-		}
-		args = argsNew
-	}
-}
-
-func delArg(args []argsKV, key []byte) []argsKV {
 	for i, n := 0, len(args); i < n; i++ {
 		kv := &args[i]
 		if bytes.Equal(kv.key, key) {
 			tmp := *kv
 			copy(args[i:], args[i+1:])
-			args[n-1] = tmp
-			return args[:n-1]
+			n--
+			args[n] = tmp
+			args = args[:n]
 		}
 	}
 	return args