chore(rad): cargo clippy

Author: guidiaz

rad/src/operators/string.rs

@@ -40,7 +40,7 @@ pub fn as_bytes(input: &RadonString, args: &Option<Vec<Value>>) -> Result<RadonB
     let wrong_args = || RadError::WrongArguments {
         input_type: RadonString::radon_type_name(),
         operator: "AsBytes".to_string(),
-        args: args.to_owned().unwrap_or(Vec::<Value>::default()).to_vec(),
+        args: args.to_owned().unwrap_or_default().to_vec(),
     };
     let mut input_string = input.value();
     if input_string.starts_with("0x") {
@@ -50,16 +50,13 @@ pub fn as_bytes(input: &RadonString, args: &Option<Vec<Value>>) -> Result<RadonB
         input_string.insert(0, '0');
     }
     let mut bytes_encoding = RadonBytesEncoding::Hex;
-    match args {
-        Some(args) => {
-            if args.len() > 0 {
-                let arg = args.first().ok_or_else(wrong_args)?.to_owned();
-                let bytes_encoding_u8 = from_value::<u8>(arg).map_err(|_| wrong_args())?;
-                bytes_encoding =
-                    RadonBytesEncoding::try_from(bytes_encoding_u8).map_err(|_| wrong_args())?;
-            }
+    if let Some(args) = args {
+        if !args.is_empty() {
+            let arg = args.first().ok_or_else(wrong_args)?.to_owned();
+            let bytes_encoding_u8 = from_value::<u8>(arg).map_err(|_| wrong_args())?;
+            bytes_encoding =
+                RadonBytesEncoding::try_from(bytes_encoding_u8).map_err(|_| wrong_args())?;
         }
-        _ => (),
     }
     match bytes_encoding {
         RadonBytesEncoding::Hex => Ok(RadonBytes::from(

rad/src/types/string.rs

@@ -121,8 +121,7 @@ impl Operable for RadonString {
                 string_operators::as_bool(self).map(RadonTypes::from)
             }
             (RadonOpCodes::StringAsBytes, args) => string_operators::as_bytes(self, args)
-                .map(RadonTypes::from)
-                .map_err(Into::into),
+                .map(RadonTypes::from),
             (RadonOpCodes::StringLength, None) => {
                 Ok(RadonTypes::from(string_operators::length(self)))
             }
@@ -140,19 +139,13 @@ impl Operable for RadonString {
             }
             (RadonOpCodes::StringReplace, Some(args)) => {
                 string_operators::string_replace(self, args.as_slice())
-                    .map(RadonTypes::from)
-                    .map_err(Into::into)
-            }
+                    .map(RadonTypes::from)}
             (RadonOpCodes::StringSlice, Some(args)) => {
                 string_operators::string_slice(self, args.as_slice())
-                    .map(RadonTypes::from)
-                    .map_err(Into::into)
-            }
+                    .map(RadonTypes::from)}
             (RadonOpCodes::StringSplit, Some(args)) => {
                 string_operators::string_split(self, args.as_slice())
-                    .map(RadonTypes::from)
-                    .map_err(Into::into)
-            }
+                    .map(RadonTypes::from)}
             (RadonOpCodes::StringToLowerCase, None) => {
                 Ok(RadonTypes::from(string_operators::to_lowercase(self)))
             }