Address review feedback

Author: setzer22

gdnative-derive/src/methods.rs

@@ -83,10 +83,7 @@ pub(crate) fn derive_methods(item_impl: ItemImpl) -> TokenStream2 {
             let sig_span = sig.ident.span();
 
             let name = sig.ident;
-            let name_string = match args.name_override {
-                Some(name_override) => name_override,
-                None => name.to_string(),
-            };
+            let name_string = args.name_override.unwrap_or_else(|| name.to_string());
             let ret_span = sig.output.span();
             let ret_ty = match sig.output {
                 syn::ReturnType::Default => quote_spanned!(ret_span => ()),
@@ -300,26 +297,12 @@ fn impl_gdnative_expose(ast: ItemImpl) -> (ItemImpl, ClassMethodExport) {
                                                 return false;
                                             };
 
-                                            // NOTE: We take advantage of rust identifiers following
-                                            // the same syntax rules as Godot method identifiers. See:
-                                            // https://docs.godotengine.org/en/stable/getting_started/scripting/gdscript/gdscript_basics.html#identifiers
-                                            match syn::parse_str::<syn::Ident>(value.as_ref()) {
-                                                Ok(_) => {
-                                                    if name_override.replace(value).is_some() {
-                                                        errors.push(syn::Error::new(
-                                                            last.span(),
-                                                            "name was set more than once",
-                                                        ));
-                                                        return false;
-                                                    }
-                                                }
-                                                Err(_) => {
-                                                    errors.push(syn::Error::new(
-                                                        last.span(),
-                                                        "name value must be a valid identifier",
-                                                    ));
-                                                    return false;
-                                                }
+                                            if name_override.replace(value).is_some() {
+                                                errors.push(syn::Error::new(
+                                                    last.span(),
+                                                    "name was set more than once",
+                                                ));
+                                                return false;
                                             }
                                         }
                                         _ => {