refactor: Drop redundant closures

locka99 · Jun 18, 2024 · 8bee67e · 8bee67e
1 parent b6648bb
commit 8bee67e
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 5 deletions.
diff --git a/lib/src/client/session/services/attributes.rs b/lib/src/client/session/services/attributes.rs
@@ -271,7 +271,7 @@ impl Session {
             // Turn the enums into ExtensionObjects
             let history_update_details = history_update_details
                 .iter()
-                .map(|action| ExtensionObject::from(action))
+                .map(ExtensionObject::from)
                 .collect::<Vec<ExtensionObject>>();
 
             let request = HistoryUpdateRequest {

diff --git a/lib/src/types/service_types/impls.rs b/lib/src/types/service_types/impls.rs
@@ -515,7 +515,7 @@ impl<'de> Deserialize<'de> for DataSetFieldFlags {
     {
         deserializer
             .deserialize_i16(Int16Visitor)
-            .map(|v| DataSetFieldFlags::from_bits_truncate(v))
+            .map(DataSetFieldFlags::from_bits_truncate)
     }
 }
 

diff --git a/lib/src/types/variant_json.rs b/lib/src/types/variant_json.rs
@@ -452,7 +452,7 @@ impl<'de> serde::de::Visitor<'de> for VariantVisitor {
                     if v.is_null() {
                         Ok(Variant::String(UAString::null()))
                     } else {
-                        json_as_value(body, "UAString").map(|v| Variant::String(v))
+                        json_as_value(body, "UAString").map(Variant::String)
                     }
                 } else {
                     Ok(Variant::String(UAString::null()))
@@ -469,14 +469,14 @@ impl<'de> serde::de::Visitor<'de> for VariantVisitor {
                     if v.is_null() {
                         Ok(Variant::ByteString(ByteString::null()))
                     } else {
-                        json_as_value(body, "ByteString").map(|v| Variant::ByteString(v))
+                        json_as_value(body, "ByteString").map(Variant::ByteString)
                     }
                 } else {
                     Ok(Variant::ByteString(ByteString::null()))
                 }
             }
             t if t == VariantJsonId::XmlElement as u32 => {
-                json_as_value(body, "XmlElement").map(|v| Variant::XmlElement(v))
+                json_as_value(body, "XmlElement").map(Variant::XmlElement)
             }
             t if t == VariantJsonId::NodeId as u32 => {
                 json_as_value(body, "NodeId").map(|v| Variant::NodeId(Box::new(v)))