Merge pull request #7 from jucr-io/method-no-return-val

Handle method w/o return values
jucr-io · Jul 29, 2024 · b4e0d76 · b4e0d76
2 parents 8cabed4 + 150b3d3
commit b4e0d76
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 7 deletions.
diff --git a/README.md b/README.md
@@ -63,7 +63,11 @@ impl Controller {
         self.set_state(State::Disabled).await;
 
         Ok(())
-    }   
+    }
+
+    // Method that doesn't return anything.
+    pub async fn return_nothing(&self) {
+    }
 }
 
 #[embassy_executor::main]

diff --git a/src/controller/item_impl.rs b/src/controller/item_impl.rs
@@ -184,12 +184,7 @@ impl ProxiedMethodArgs<'_> {
         let in_args = MethodInputArgs::parse(method)?;
         let out_type = match &method.sig.output {
             syn::ReturnType::Type(_, ty) => quote! { #ty },
-            syn::ReturnType::Default => {
-                return Err(syn::Error::new(
-                    method.sig.ident.span(),
-                    "Expected return type in method signature",
-                ))
-            }
+            syn::ReturnType::Default => quote! { () },
         };
 
         Ok(ProxiedMethodArgs {