Merge pull request #393 from matthiasbeyer/more-clippy-fixes

More clippy fixes

Author: matthiasbeyer

src/env.rs

@@ -140,7 +140,7 @@ impl Environment {
     /// Once list_separator is set, the type for string is [`Vec<String>`].
     /// To switch the default type back to type Strings you need to provide the keys which should be [`Vec<String>`] using this function.
     pub fn with_list_parse_key(mut self, key: &str) -> Self {
-        if self.list_parse_keys == None {
+        if self.list_parse_keys.is_none() {
             self.list_parse_keys = Some(vec![key.into()])
         } else {
             self.list_parse_keys = self.list_parse_keys.map(|mut keys| {

src/value.rs

@@ -309,7 +309,7 @@ impl Value {
                 }
             }
 
-            ValueKind::Boolean(value) => Ok(if value { 1 } else { 0 }),
+            ValueKind::Boolean(value) => Ok(i64::from(value)),
             ValueKind::Float(value) => Ok(value.round() as i64),
 
             // Unexpected type
@@ -362,7 +362,7 @@ impl Value {
                 }
             }
 
-            ValueKind::Boolean(value) => Ok(if value { 1 } else { 0 }),
+            ValueKind::Boolean(value) => Ok(i128::from(value)),
             ValueKind::Float(value) => Ok(value.round() as i128),
 
             // Unexpected type
@@ -428,7 +428,7 @@ impl Value {
                 }
             }
 
-            ValueKind::Boolean(value) => Ok(if value { 1 } else { 0 }),
+            ValueKind::Boolean(value) => Ok(u64::from(value)),
             ValueKind::Float(value) => Ok(value.round() as u64),
 
             // Unexpected type
@@ -487,7 +487,7 @@ impl Value {
                 }
             }
 
-            ValueKind::Boolean(value) => Ok(if value { 1 } else { 0 }),
+            ValueKind::Boolean(value) => Ok(u128::from(value)),
             ValueKind::Float(value) => Ok(value.round() as u128),
 
             // Unexpected type