Bump version for release and a few clippy warnings fixed

Author: benashford

Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "redis-async"
-version = "0.4.2"
+version = "0.4.3"
 authors = ["Ben Ashford <[email protected]>"]
 license = "MIT/Apache-2.0"
 readme = "README.md"

examples/monitor.rs

@@ -22,7 +22,7 @@ use redis_async::client;
 fn main() {
     let addr = env::args()
         .nth(1)
-        .unwrap_or("127.0.0.1:6379".to_string())
+        .unwrap_or_else(|| "127.0.0.1:6379".to_string())
         .parse()
         .unwrap();
 

examples/pubsub.rs

@@ -20,10 +20,12 @@ use redis_async::client;
 use redis_async::resp::FromResp;
 
 fn main() {
-    let topic = env::args().nth(1).unwrap_or("test-topic".to_string());
+    let topic = env::args()
+        .nth(1)
+        .unwrap_or_else(|| "test-topic".to_string());
     let addr = env::args()
         .nth(2)
-        .unwrap_or("127.0.0.1:6379".to_string())
+        .unwrap_or_else(|| "127.0.0.1:6379".to_string())
         .parse()
         .unwrap();
 

examples/realistic.rs

@@ -29,7 +29,7 @@ fn main() {
 
     let addr = env::args()
         .nth(1)
-        .unwrap_or("127.0.0.1:6379".to_string())
+        .unwrap_or_else(|| "127.0.0.1:6379".to_string())
         .parse()
         .unwrap();
 

src/resp.rs

@@ -11,7 +11,7 @@
 //! An implementation of the RESP protocol
 
 use std::collections::HashMap;
-use std::hash::Hash;
+use std::hash::{BuildHasher, Hash};
 use std::io;
 use std::str;
 
@@ -189,19 +189,21 @@ impl<T: FromResp> FromResp for Vec<T> {
     }
 }
 
-impl<K: FromResp + Hash + Eq, T: FromResp> FromResp for HashMap<K, T> {
-    fn from_resp_int(resp: RespValue) -> Result<HashMap<K, T>, Error> {
+impl<K: FromResp + Hash + Eq, T: FromResp, S: BuildHasher + Default> FromResp for HashMap<K, T, S> {
+    fn from_resp_int(resp: RespValue) -> Result<HashMap<K, T, S>, Error> {
         match resp {
             RespValue::Array(ary) => {
-                let mut map = HashMap::new();
+                let mut map = HashMap::with_capacity_and_hasher(ary.len(), S::default());
                 let mut items = ary.into_iter();
 
                 while let Some(k) = items.next() {
                     let key = K::from_resp(k)?;
-                    let value = T::from_resp(items.next().ok_or(error::resp(
-                        "Cannot convert an odd number of elements into a hashmap",
-                        "".into(),
-                    ))?)?;
+                    let value = T::from_resp(items.next().ok_or_else(|| {
+                        error::resp(
+                            "Cannot convert an odd number of elements into a hashmap",
+                            "".into(),
+                        )
+                    })?)?;
 
                     map.insert(key, value);
                 }