perf(param): ignore empty args from config

marjune163 · marjune163 · commit 97ced9f9a7cf · 2025-02-14T23:41:53.000+08:00
diff --git a/src/param/cli.go b/src/param/cli.go
@@ -265,7 +265,7 @@ func ArgsToCmdResults(cmd *goNixArgParser.Command, args []string) (results []*go
 	}
 
 	// append config and re-parse
-	configs := []string{}
+	configs := make([]string, 0, len(results))
 	groupSeps := cmd.Options().GroupSeps()[0]
 	hasConfig := false
 	var stdinConfigArgs []string
@@ -292,6 +292,9 @@ func ArgsToCmdResults(cmd *goNixArgParser.Command, args []string) (results []*go
 				stdinConfigArgs = configArgs
 			}
 		}
+		if len(configArgs) == 0 {
+			continue
+		}
 
 		hasConfig = true
 		configs = append(configs, configArgs...)

Original file line number	Diff line number	Diff line change
`@@ -265,7 +265,7 @@ func ArgsToCmdResults(cmd goNixArgParser.Command, args []string) (results []go`
`265`	`265`	`}`
`266`	`266`
`267`	`267`	`// append config and re-parse`
`268`		`- configs := []string{}`
	`268`	`+ configs := make([]string, 0, len(results))`
`269`	`269`	`groupSeps := cmd.Options().GroupSeps()[0]`
`270`	`270`	`hasConfig := false`
`271`	`271`	`var stdinConfigArgs []string`
`@@ -292,6 +292,9 @@ func ArgsToCmdResults(cmd goNixArgParser.Command, args []string) (results []go`
`292`	`292`	`stdinConfigArgs = configArgs`
`293`	`293`	`}`
`294`	`294`	`}`
	`295`	`+ if len(configArgs) == 0 {`
	`296`	`+ continue`
	`297`	`+ }`
`295`	`298`
`296`	`299`	`hasConfig = true`
`297`	`300`	`configs = append(configs, configArgs...)`