NaxdyOrg/embassy
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Merge pull request #12 from timokroeger/task-macro-warning

Browse source 
Fix warnings for tasks that take mutable parameters
This commit is contained in:
Dario Nieuwenhuis 2020-12-31 16:32:44 +01:00 committed by GitHub
commit ffb92731cd
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 7 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
embassy-macros/src/lib.rs
View file

@ -55,9 +55,9 @@ pub fn task(args: TokenStream, item: TokenStream) -> TokenStream {
let mut arg_names: syn::punctuated::Punctuated<syn::Ident, syn::Token![,]> = let mut arg_names: syn::punctuated::Punctuated<syn::Ident, syn::Token![,]> =
syn::punctuated::Punctuated::new(); syn::punctuated::Punctuated::new();
let args = &task_fn.sig.inputs; let mut args = task_fn.sig.inputs.clone();
for arg in args.iter() { for arg in args.iter_mut() {
match arg { match arg {
syn::FnArg::Receiver(_) => { syn::FnArg::Receiver(_) => {
arg.span() arg.span()
@ -66,8 +66,11 @@ pub fn task(args: TokenStream, item: TokenStream) -> TokenStream {
.emit(); .emit();
fail = true; fail = true;
} }
syn::FnArg::Typed(t) => match t.pat.as_ref() { syn::FnArg::Typed(t) => match t.pat.as_mut() {
syn::Pat::Ident(i) => arg_names.push(i.ident.clone()), syn::Pat::Ident(i) => {
arg_names.push(i.ident.clone());
i.mutability = None;
}
_ => { _ => {
arg.span() arg.span()
.unwrap() .unwrap()