Lazily evaluate error string (paritytech#7802)

JoshOrndorff · web-flow · commit 264cfcf8e199 · 2020-12-29T07:51:17.000Z
diff --git a/bin/node-template/node/src/chain_spec.rs b/bin/node-template/node/src/chain_spec.rs
@@ -39,7 +39,7 @@ pub fn authority_keys_from_seed(s: &str) -> (AuraId, GrandpaId) {
 }
 
 pub fn development_config() -> Result<ChainSpec, String> {
-	let wasm_binary = WASM_BINARY.ok_or("Development wasm binary not available".to_string())?;
+	let wasm_binary = WASM_BINARY.ok_or_else(|| "Development wasm not available".to_string())?;
 
 	Ok(ChainSpec::from_genesis(
 		// Name
@@ -78,7 +78,7 @@ pub fn development_config() -> Result<ChainSpec, String> {
 }
 
 pub fn local_testnet_config() -> Result<ChainSpec, String> {
-	let wasm_binary = WASM_BINARY.ok_or("Development wasm binary not available".to_string())?;
+	let wasm_binary = WASM_BINARY.ok_or_else(|| "Development wasm not available".to_string())?;
 
 	Ok(ChainSpec::from_genesis(
 		// Name

Original file line number	Diff line number	Diff line change
`@@ -39,7 +39,7 @@ pub fn authority_keys_from_seed(s: &str) -> (AuraId, GrandpaId) {`
`39`	`39`	`}`
`40`	`40`
`41`	`41`	`pub fn development_config() -> Result<ChainSpec, String> {`
`42`		`- let wasm_binary = WASM_BINARY.ok_or("Development wasm binary not available".to_string())?;`
	`42`	`+ let wasm_binary = WASM_BINARY.ok_or_else(\|\| "Development wasm not available".to_string())?;`
`43`	`43`
`44`	`44`	`Ok(ChainSpec::from_genesis(`
`45`	`45`	`// Name`
`@@ -78,7 +78,7 @@ pub fn development_config() -> Result<ChainSpec, String> {`
`78`	`78`	`}`
`79`	`79`
`80`	`80`	`pub fn local_testnet_config() -> Result<ChainSpec, String> {`
`81`		`- let wasm_binary = WASM_BINARY.ok_or("Development wasm binary not available".to_string())?;`
	`81`	`+ let wasm_binary = WASM_BINARY.ok_or_else(\|\| "Development wasm not available".to_string())?;`
`82`	`82`
`83`	`83`	`Ok(ChainSpec::from_genesis(`
`84`	`84`	`// Name`