Add ref t into wasm (#63)

Derive IntoWasmAbi for &T

Previously `IntoWasmAbi T` was derived, but not `IntoWasmAbi &T`

Author: cmrschwarz

tests-e2e/reference_output/test5/test5.d.ts

@@ -0,0 +1,8 @@
+/* tslint:disable */
+/* eslint-disable */
+export function accept_ref_point(point: Point): void;
+export interface Point {
+    x: number;
+    y: number;
+}
+

tests-e2e/test5/Cargo.toml

@@ -0,0 +1,21 @@
+[package]
+name = "test5"
+publish = false
+version = "0.1.0"
+edition = "2021"
+
+[dependencies]
+wasm-bindgen = "0.2"
+tsify = { path = "../..", version = "*" }
+serde = { version = "1.0", features = ["derive"] }
+serde_json = "1.0"
+
+[dev-dependencies]
+wasm-bindgen-test = "0.3"
+
+[lib]
+path = "entry_point.rs"
+crate-type = ["cdylib"]
+
+[build-dependencies]
+wasm-bindgen-cli = "0.2"

tests-e2e/test5/entry_point.rs

@@ -0,0 +1,19 @@
+use serde::{Deserialize, Serialize};
+use tsify::Tsify;
+use wasm_bindgen::prelude::*;
+
+#[derive(Tsify, Serialize, Deserialize)]
+#[tsify(into_wasm_abi, from_wasm_abi)]
+pub struct Point {
+    x: i32,
+    y: i32,
+}
+
+#[wasm_bindgen]
+extern "C" {
+    #[wasm_bindgen]
+    pub fn call_js(point: &Point);
+}
+
+#[wasm_bindgen]
+pub fn accept_ref_point(point: &Point) {}

tests/expand/borrow.expanded.rs

@@ -47,7 +47,7 @@ const _: () = {
         }
     }
     #[automatically_derived]
-    impl<'a> IntoWasmAbi for Borrow<'a>
+    impl<'a> IntoWasmAbi for &Borrow<'a>
     where
         Borrow<'a>: _serde::Serialize,
     {
@@ -85,6 +85,17 @@ const _: () = {
         }
     }
     #[automatically_derived]
+    impl<'a> IntoWasmAbi for Borrow<'a>
+    where
+        Borrow<'a>: _serde::Serialize,
+    {
+        type Abi = <JsType as IntoWasmAbi>::Abi;
+        #[inline]
+        fn into_abi(self) -> Self::Abi {
+            (&self).into_abi()
+        }
+    }
+    #[automatically_derived]
     impl<'a> OptionIntoWasmAbi for Borrow<'a>
     where
         Borrow<'a>: _serde::Serialize,

tests/expand/generic_enum.expanded.rs

@@ -48,7 +48,7 @@ const _: () = {
         }
     }
     #[automatically_derived]
-    impl<T, U> IntoWasmAbi for GenericEnum<T, U>
+    impl<T, U> IntoWasmAbi for &GenericEnum<T, U>
     where
         GenericEnum<T, U>: _serde::Serialize,
     {
@@ -86,6 +86,17 @@ const _: () = {
         }
     }
     #[automatically_derived]
+    impl<T, U> IntoWasmAbi for GenericEnum<T, U>
+    where
+        GenericEnum<T, U>: _serde::Serialize,
+    {
+        type Abi = <JsType as IntoWasmAbi>::Abi;
+        #[inline]
+        fn into_abi(self) -> Self::Abi {
+            (&self).into_abi()
+        }
+    }
+    #[automatically_derived]
     impl<T, U> OptionIntoWasmAbi for GenericEnum<T, U>
     where
         GenericEnum<T, U>: _serde::Serialize,

tests/expand/generic_struct.expanded.rs

@@ -45,7 +45,7 @@ const _: () = {
         }
     }
     #[automatically_derived]
-    impl<T> IntoWasmAbi for GenericStruct<T>
+    impl<T> IntoWasmAbi for &GenericStruct<T>
     where
         GenericStruct<T>: _serde::Serialize,
     {
@@ -83,6 +83,17 @@ const _: () = {
         }
     }
     #[automatically_derived]
+    impl<T> IntoWasmAbi for GenericStruct<T>
+    where
+        GenericStruct<T>: _serde::Serialize,
+    {
+        type Abi = <JsType as IntoWasmAbi>::Abi;
+        #[inline]
+        fn into_abi(self) -> Self::Abi {
+            (&self).into_abi()
+        }
+    }
+    #[automatically_derived]
     impl<T> OptionIntoWasmAbi for GenericStruct<T>
     where
         GenericStruct<T>: _serde::Serialize,
@@ -284,7 +295,7 @@ const _: () = {
         }
     }
     #[automatically_derived]
-    impl<T> IntoWasmAbi for GenericNewtype<T>
+    impl<T> IntoWasmAbi for &GenericNewtype<T>
     where
         GenericNewtype<T>: _serde::Serialize,
     {
@@ -322,6 +333,17 @@ const _: () = {
         }
     }
     #[automatically_derived]
+    impl<T> IntoWasmAbi for GenericNewtype<T>
+    where
+        GenericNewtype<T>: _serde::Serialize,
+    {
+        type Abi = <JsType as IntoWasmAbi>::Abi;
+        #[inline]
+        fn into_abi(self) -> Self::Abi {
+            (&self).into_abi()
+        }
+    }
+    #[automatically_derived]
     impl<T> OptionIntoWasmAbi for GenericNewtype<T>
     where
         GenericNewtype<T>: _serde::Serialize,

tests/wasm.rs

@@ -31,17 +31,17 @@ struct SimpleData {
 
 #[wasm_bindgen(inline_js = r#"
 function validate(value, validation) {
-    validation(value); 
+    validation(value);
 
     // Validate twice to make sure the value is not moved in any way to rust
-    validation(value); 
+    validation(value);
 }
 
 function validateArray(value, validation) {
     validation(value);
 
     // Validate twice to make sure the value is not moved in any way to rust
-    validation(value); 
+    validation(value);
 }
 
 function noop(value) {}
@@ -59,7 +59,7 @@ extern "C" {
 
     #[wasm_bindgen(catch, js_name = "validate")]
     pub fn validate_simple_data_ref(
-        value: SimpleData,
+        value: &SimpleData,
         validation: &dyn Fn(&SimpleData),
     ) -> Result<(), JsValue>;
 
@@ -98,7 +98,7 @@ fn test_convert_simple_value_type() {
     })
     .unwrap_throw();
 
-    validate_simple_data_ref(first_value.clone(), &|val_after| {
+    validate_simple_data_ref(&first_value, &|val_after| {
         assert_eq!(val_after, &first_value);
     })
     .unwrap_throw();

tsify-macros/src/wasm_bindgen.rs

@@ -107,7 +107,7 @@ fn expand_into_wasm_abi(cont: &Container) -> TokenStream {
 
     quote! {
         #[automatically_derived]
-        impl #impl_generics IntoWasmAbi for #ident #ty_generics #where_clause {
+        impl #impl_generics IntoWasmAbi for &#ident #ty_generics #where_clause {
             type Abi = <JsType as IntoWasmAbi>::Abi;
 
             #[inline]
@@ -129,6 +129,16 @@ fn expand_into_wasm_abi(cont: &Container) -> TokenStream {
             }
         }
 
+        #[automatically_derived]
+        impl #impl_generics IntoWasmAbi for #ident #ty_generics #where_clause {
+            type Abi = <JsType as IntoWasmAbi>::Abi;
+
+            #[inline]
+            fn into_abi(self) -> Self::Abi {
+                (&self).into_abi()
+            }
+        }
+
         #[automatically_derived]
         impl #impl_generics OptionIntoWasmAbi for #ident #ty_generics #where_clause {
             #[inline]