[snowpatch] [PATCH] clippy warnings: use of assert_eq!()

Andrew Donnellan andrew.donnellan at au1.ibm.com
Thu Mar 23 12:48:50 AEDT 2017


Change a couple of assert!()s to assert_eq!() for additional clarity.

Signed-off-by: Andrew Donnellan <andrew.donnellan at au1.ibm.com>

---

Applied to master: c869d952f69e813eb035fa2a7a1672703444008e

---
 src/main.rs | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/main.rs b/src/main.rs
index 7941a90..6ba550e 100644
--- a/src/main.rs
+++ b/src/main.rs
@@ -277,9 +277,9 @@ fn main() {
                 panic!("http_proxy is malformed: {:?}, error: {}", proxy_url, e);
             });
             assert!(proxy.has_host());
-            assert!(proxy.scheme() == "http");
+            assert_eq!(proxy.scheme(), "http");
             // This should pass even if no trailing slash is in http_proxy
-            assert!(proxy.path() == "/");
+            assert_eq!(proxy.path(), "/");
             let proxy_config = ProxyConfig::new(proxy.scheme(),
                                                 proxy.host_str().unwrap().to_string(),
                                                 proxy.port().unwrap_or(80),
-- 
Andrew Donnellan              OzLabs, ADL Canberra
andrew.donnellan at au1.ibm.com  IBM Australia Limited



More information about the snowpatch mailing list