Don't call functions in .unwrap_or.

tomprince · tomprince · commit ac8d96d3a8c1 · 2017-01-24T11:41:27.000-07:00
diff --git a/tests/helpers.rs b/tests/helpers.rs
@@ -4,6 +4,6 @@ use std::env;
 use std::net::TcpStream;
 
 pub fn connect() -> mpd::Client<TcpStream> {
-    let addr = env::var("MPD_SOCK").unwrap_or("127.0.0.1:6600".to_owned());
+    let addr = env::var("MPD_SOCK").unwrap_or_else(|_| "127.0.0.1:6600".to_owned());
     mpd::Client::connect(&*addr).unwrap()
 }

Original file line number	Diff line number	Diff line change
`@@ -4,6 +4,6 @@ use std::env;`
`4`	`4`	`use std::net::TcpStream;`
`5`	`5`
`6`	`6`	`pub fn connect() -> mpd::Client<TcpStream> {`
`7`		`- let addr = env::var("MPD_SOCK").unwrap_or("127.0.0.1:6600".to_owned());`
	`7`	`+ let addr = env::var("MPD_SOCK").unwrap_or_else(\|_\| "127.0.0.1:6600".to_owned());`
`8`	`8`	`mpd::Client::connect(&*addr).unwrap()`
`9`	`9`	`}`