cli/command/system: prettyPrintServerInfo: refactor printing "runtimes"

thaJeztah · thaJeztah · commit be6f4cd56c89 · 2023-05-02T16:05:52.000+02:00
Signed-off-by: Sebastiaan van Stijn &lt;github@gone.nl&gt;
diff --git a/cli/command/system/info.go b/cli/command/system/info.go
@@ -253,11 +253,11 @@ func prettyPrintServerInfo(dockerCli command.Cli, info *info) []error {
 	printSwarmInfo(dockerCli, *info.Info)
 
 	if len(info.Runtimes) > 0 {
-		fmt.Fprint(dockerCli.Out(), " Runtimes:")
+		names := make([]string, 0, len(info.Runtimes))
 		for name := range info.Runtimes {
-			fmt.Fprintf(dockerCli.Out(), " %s", name)
+			names = append(names, name)
 		}
-		fmt.Fprint(dockerCli.Out(), "\n")
+		fmt.Fprintln(dockerCli.Out(), " Runtimes:", strings.Join(names, " "))
 		fmt.Fprintln(dockerCli.Out(), " Default Runtime:", info.DefaultRuntime)
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -253,11 +253,11 @@ func prettyPrintServerInfo(dockerCli command.Cli, info *info) []error {`
`253`	`253`	`printSwarmInfo(dockerCli, *info.Info)`
`254`	`254`
`255`	`255`	`if len(info.Runtimes) > 0 {`
`256`		`- fmt.Fprint(dockerCli.Out(), " Runtimes:")`
	`256`	`+ names := make([]string, 0, len(info.Runtimes))`
`257`	`257`	`for name := range info.Runtimes {`
`258`		`- fmt.Fprintf(dockerCli.Out(), " %s", name)`
	`258`	`+ names = append(names, name)`
`259`	`259`	`}`
`260`		`- fmt.Fprint(dockerCli.Out(), "\n")`
	`260`	`+ fmt.Fprintln(dockerCli.Out(), " Runtimes:", strings.Join(names, " "))`
`261`	`261`	`fmt.Fprintln(dockerCli.Out(), " Default Runtime:", info.DefaultRuntime)`
`262`	`262`	`}`
`263`	`263`