diff --git a/src/bindings.rs b/src/bindings.rs index 474c211..dc2b49c 100644 --- a/src/bindings.rs +++ b/src/bindings.rs @@ -638,6 +638,7 @@ fn generate_random_number( if args.is_empty() { Ok(rand.gen::()) } else if args.len() == 1 { + #[allow(clippy::get_first)] let max = args.get(0).unwrap().as_integer(); if let Some(max) = max { if max >= 1 { @@ -660,6 +661,7 @@ fn generate_random_number( )) } } else if args.len() == 2 { + #[allow(clippy::get_first)] let min = args.get(0).unwrap().as_integer(); let max = args.get(1).unwrap().as_integer(); if let Some(min) = min { diff --git a/src/bindings/callback.rs b/src/bindings/callback.rs index 95d4856..e7011d1 100644 --- a/src/bindings/callback.rs +++ b/src/bindings/callback.rs @@ -303,7 +303,7 @@ impl LuaCallback { let result = self .function .call::((&self.context, arg.clone()))?; - if let Some(inner_function) = result.as_function().cloned().map(|f| f) { + if let Some(inner_function) = result.as_function().cloned() { // function returned a function -> is a generator. use the inner function instead. let environment = self.function.environment(); self.environment = environment; diff --git a/src/bindings/unwrap.rs b/src/bindings/unwrap.rs index 47bc4da..1a7e87a 100644 --- a/src/bindings/unwrap.rs +++ b/src/bindings/unwrap.rs @@ -111,7 +111,7 @@ pub(crate) fn string_from_value( arg_index: usize, ) -> LuaResult { if let Some(string) = value.as_string_lossy() { - Ok(string.into()) + Ok(string) } else { Err(bad_argument_error( function, @@ -130,7 +130,7 @@ pub(crate) fn optional_string_from_value( arg_index: usize, ) -> LuaResult { if let Some(string) = value.as_string_lossy() { - Ok(string.into()) + Ok(string) } else if value.is_nil() { Ok(String::new()) } else {