chore: cargo clippy --fix

Author: guidiaz

rad/src/operators/array.rs

@@ -118,7 +118,7 @@ pub fn join(
             operator: "ArrayJoin".to_string(),
         });
     }
-    let separator = if args.len() > 0 {
+    let separator = if !args.is_empty() {
         from_value::<String>(args[0].to_owned()).unwrap_or_default()
     } else {
         String::from("")
@@ -131,14 +131,14 @@ pub fn join(
             Ok(RadonTypes::from(RadonString::from(string_list.join(separator.as_str()))))
         }
         Some(first_item) => {
-            return Err(RadError::UnsupportedOperator { 
+            Err(RadError::UnsupportedOperator { 
                 input_type: first_item.radon_type_name().to_string(),
                 operator: "ArrayJoin".to_string(), 
                 args: Some(args.to_vec())
-            });
+            })
         }
         _ => {
-            return Err(RadError::EmptyArray)
+            Err(RadError::EmptyArray)
         }
     }
 }

rad/src/operators/bytes.rs

@@ -66,12 +66,12 @@ pub fn stringify(input: &RadonBytes, args: &Option<Vec<Value>>) -> Result<RadonS
     let wrong_args = || RadError::WrongArguments { 
         input_type: RadonString::radon_type_name(),
         operator: "Stringify".to_string(),
-        args: args.to_owned().unwrap_or(Vec::<Value>::default()).to_vec(),
+        args: args.to_owned().unwrap_or_default().to_vec(),
     };
     let mut bytes_encoding = RadonBytesEncoding::Hex;
     match args {
         Some(args) => {
-            if args.len() > 0 {
+            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())?;

rad/src/operators/string.rs

@@ -37,7 +37,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") {
@@ -49,7 +49,7 @@ pub fn as_bytes(input: &RadonString, args: &Option<Vec<Value>>) -> Result<RadonB
     let mut bytes_encoding = RadonBytesEncoding::Hex;
     match args {
         Some(args) => {
-            if args.len() > 0 {
+            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())?;
@@ -203,7 +203,7 @@ pub fn parse_json_array(input: &RadonString, args: &Option<Vec<Value>>) -> Resul
                             description: err.to_string(),
                         })?;
                     let mut subitems: Vec<RadonTypes> = selector.find(&json_input)
-                        .map(|item| into_radon_types(item))
+                        .map(into_radon_types)
                         .collect();
                     if subitems.len() > 1 {
                         items.insert(items.len(), RadonArray::from(subitems).into());
@@ -222,7 +222,7 @@ pub fn parse_json_array(input: &RadonString, args: &Option<Vec<Value>>) -> Resul
                     description: err.to_string(),
                 })?;
             let items: Vec<RadonTypes> = selector.find(&json_input)
-                .map(|item| into_radon_types(item))
+                .map(into_radon_types)
                 .collect();
             Ok(RadonArray::from(items))
         }
@@ -254,7 +254,7 @@ fn into_radon_types(value: &serde_json::Value) -> RadonTypes {
         serde_json::Value::Array(values) => {
             let items: Vec<RadonTypes> = values
                 .iter()
-                .map(|item| into_radon_types(item))
+                .map(into_radon_types)
                 .collect();
             RadonTypes::from(RadonArray::from(items))
         }

rad/src/types/string.rs

@@ -131,10 +131,10 @@ impl Operable for RadonString {
             (RadonOpCodes::StringMatch, Some(args)) => {
                 string_operators::string_match(self, args.as_slice()).map(RadonTypes::from)
             }
-            (RadonOpCodes::StringParseJSONArray, args) => string_operators::parse_json_array(self, &args)
+            (RadonOpCodes::StringParseJSONArray, args) => string_operators::parse_json_array(self, args)
                 .map(RadonTypes::from)
                 .map_err(Into::into),
-            (RadonOpCodes::StringParseJSONMap, args) => string_operators::parse_json_map(self, &args)
+            (RadonOpCodes::StringParseJSONMap, args) => string_operators::parse_json_map(self, args)
                 .map(RadonTypes::from)
                 .map_err(Into::into),
             (RadonOpCodes::StringParseXMLMap, None) => string_operators::parse_xml_map(self)