From 7e7a45ac198edc0f2fce257f194c8ab8c1d87c3f Mon Sep 17 00:00:00 2001
From: Ed Page <eopage@gmail.com>
Date: Sat, 18 Mar 2023 14:51:13 -0500
Subject: [PATCH] perf(derive): Reduce the amount of generated code

Its a small win, dropping the number of copies and reducing the
overhead from `quote` but enough of these could make a difference
---
 clap_derive/src/item.rs | 14 +++++++++-----
 1 file changed, 9 insertions(+), 5 deletions(-)

diff --git a/clap_derive/src/item.rs b/clap_derive/src/item.rs
index b7814f7d..22df4b60 100644
--- a/clap_derive/src/item.rs
+++ b/clap_derive/src/item.rs
@@ -277,6 +277,10 @@ impl Item {
     }
 
     fn push_method(&mut self, kind: AttrKind, name: Ident, arg: impl ToTokens) {
+        self.push_method_(kind, name, arg.to_token_stream());
+    }
+
+    fn push_method_(&mut self, kind: AttrKind, name: Ident, arg: TokenStream) {
         if name == "id" {
             match kind {
                 AttrKind::Command | AttrKind::Value => {
@@ -293,7 +297,7 @@ impl Item {
                 }
                 AttrKind::Group | AttrKind::Arg | AttrKind::Clap | AttrKind::StructOpt => {}
             }
-            self.name = Name::Assigned(quote!(#arg));
+            self.name = Name::Assigned(arg);
         } else if name == "name" {
             match kind {
                 AttrKind::Arg => {
@@ -315,16 +319,16 @@ impl Item {
                 | AttrKind::Clap
                 | AttrKind::StructOpt => {}
             }
-            self.name = Name::Assigned(quote!(#arg));
+            self.name = Name::Assigned(arg);
         } else if name == "value_parser" {
-            self.value_parser = Some(ValueParser::Explicit(Method::new(name, quote!(#arg))));
+            self.value_parser = Some(ValueParser::Explicit(Method::new(name, arg)));
         } else if name == "action" {
-            self.action = Some(Action::Explicit(Method::new(name, quote!(#arg))));
+            self.action = Some(Action::Explicit(Method::new(name, arg)));
         } else {
             if name == "short" || name == "long" {
                 self.is_positional = false;
             }
-            self.methods.push(Method::new(name, quote!(#arg)));
+            self.methods.push(Method::new(name, arg));
         }
     }