Treat some references differently for ergonomics (#13)

Discovered by @Madoshakalaka  

If a reference type is actually intended, `#[autoprops]` will currently
ignore that and put a referenceless type into the props struct. For
example:

```rust
#[autoprops]
#[component]
pub fn Test(string: &'static str) {
    html!{}
}
```

In this case `'static` is ignored, and `str` is attempted to be put into
props struct. As a workaround, currently works:

```rust
#[autoprops]
#[component]
pub fn Test(string: &&'static str) {
    html!{}
}
```

But it is unergonomic. It is well known that reference is wanted (by the
lifetime specifier existing), so let's put such references into props
struct as-is.

Author: kirillsemyonkin

.gitignore

@@ -2,3 +2,4 @@
 output.log
 .envrc
 .aider*
+.vscode

src/autoprops.rs

@@ -119,7 +119,10 @@ impl Autoprops {
             .iter()
             .filter_map(|arg| match arg {
                 syn::FnArg::Typed(syn::PatType { pat, ty, .. })
-                    if !matches!(**ty, syn::Type::Reference(_)) =>
+                    if !matches!(
+                        **ty,
+                        syn::Type::Reference(syn::TypeReference { lifetime: None, .. })
+                    ) =>
                 {
                     Some(quote! {
                         let #pat = ::yew::html::ImplicitClone::implicit_clone(#pat);
@@ -154,9 +157,11 @@ impl Autoprops {
             .iter()
             .filter_map(|arg| match arg {
                 syn::FnArg::Typed(syn::PatType { attrs, pat, ty, .. }) => match ty.as_ref() {
-                    syn::Type::Reference(syn::TypeReference { elem, .. }) => {
-                        Some(quote! { #(#attrs)* #vis #pat: #elem, })
-                    }
+                    syn::Type::Reference(syn::TypeReference {
+                        elem,
+                        lifetime: None,
+                        ..
+                    }) => Some(quote! { #(#attrs)* #vis #pat: #elem, }),
                     _ => Some(quote! { #(#attrs)* #vis #pat: #ty, }),
                 },
                 _ => None,

src/function_component.rs

@@ -1,4 +1,5 @@
 // copy-pasted from yew-macro because proc-macro crates cannot export any items
+// TODO separate yew's macros into their own library common for yew-macro and yew-autoprops
 
 use syn::parse::{Parse, ParseStream};
 use syn::Ident;

src/lib.rs

@@ -11,6 +11,5 @@ pub fn autoprops(
     let mut autoprops = syn::parse_macro_input!(item as autoprops::Autoprops);
     let args = syn::parse_macro_input!(attr as autoprops::AutopropsArgs);
     autoprops.apply_args(args);
-
-    proc_macro::TokenStream::from(autoprops.into_token_stream())
+    autoprops.into_token_stream().into()
 }

tests/function_component_attr/autoprops-pass.rs

@@ -103,6 +103,13 @@ where
     }
 }
 
+#[::yew_autoprops::autoprops]
+#[::yew::component]
+fn CompWithIntendedRef(s: &'static ::std::primitive::str) -> ::yew::Html {
+    let _ = s;
+    ::yew::html! {}
+}
+
 fn compile_pass() {
     let _ = ::yew::html! { <CompUseFnName /> };
     let _ = ::yew::html! { <CompNoProperties /> };
@@ -115,6 +122,7 @@ fn compile_pass() {
     let _ = ::yew::html! { <CompHtmlResult /> };
     let _ = ::yew::html! { <CompWithDefaultGeneric /> };
     let _ = ::yew::html! { <CompWithDefaultGeneric<::std::primitive::u32> /> };
+    let _ = ::yew::html! { <CompWithIntendedRef s="foo" /> };
 }
 
 fn main() {}