refactor(cli): remove allocation and use format arg capture

Author: weihanglo

src/bin/cargo/cli.rs

@@ -61,36 +61,27 @@ pub fn main(config: &mut LazyConfig) -> CliResult {
         let reset = anstyle::Reset.render();
 
         let options = CliUnstable::help();
-        let non_hidden_options: Vec<(String, String)> = options
-            .iter()
-            .filter(|(_, help_message)| *help_message != HIDDEN)
-            .map(|(name, help)| (name.to_string(), help.to_string()))
-            .collect();
-        let longest_option = non_hidden_options
+        let max_length = options
             .iter()
             .map(|(option_name, _)| option_name.len())
             .max()
-            .unwrap_or(0);
-        let help_lines: Vec<String> = non_hidden_options
+            .unwrap();
+        let z_flags = options
             .iter()
-            .map(|(option_name, option_help_message)| {
-                let option_name_kebab_case = option_name.replace("_", "-");
-                format!(
-                    "    {literal}-Z {:<longest_option$}{reset}  {}",
-                    option_name_kebab_case, option_help_message
-                )
+            .filter(|(_, help)| *help != HIDDEN)
+            .map(|(opt, help)| {
+                let opt = opt.replace("_", "-");
+                format!("    {literal}-Z {opt:<max_length$}{reset}  {help}")
             })
-            .collect();
-        let joined = help_lines.join("\n");
+            .join("\n");
         drop_println!(
             config,
             "\
 {header}Available unstable (nightly-only) flags:{reset}
 
-{}
+{z_flags}
 
 Run with `{literal}cargo -Z{reset} {placeholder}[FLAG] [COMMAND]{reset}`",
-            joined
         );
         if !config.nightly_features_allowed {
             drop_println!(