Repository: aisk/rust-memcache (language: rust)

Issue:
Possible bug in version command.
In the `version` call we return a `Vec(String, String)` but the first element of the tuple is [always empty](https://github.com/aisk/rust-memcache/blob/0498345a54aeb860e8c6ac2485b09fd8ef547c9d/src/client.rs#L142)? Perhaps the intention was to do something similar to [this](https://github.com/aisk/rust-memcache/blob/0498345a54aeb860e8c6ac2485b09fd8ef547c9d/src/client.rs#L390)? 


Discussion / hints:

Fix patch:
diff --git a/src/client.rs b/src/client.rs
index 96d75e1..e491e2c 100644
--- a/src/client.rs
+++ b/src/client.rs
@@ -133,11 +133,11 @@ impl Client {
     /// client.version().unwrap();
     /// ```
     pub fn version(&mut self) -> Result<Vec<(String, String)>, MemcacheError> {
-        let mut result: Vec<(String, String)> = vec![];
+        let mut result = Vec::with_capacity(self.connections.len());
         for connection in &mut self.connections {
-            result.push(("".into(), connection.protocol.version()?));
+            result.push((connection.url.clone(), connection.protocol.version()?));
         }
-        return Ok(result);
+        Ok(result)
     }
 
     /// Flush all cache on memcached server immediately.


Test patch: