From 21d411f57b42090ee466e52ff2c41eb49a7a155b Mon Sep 17 00:00:00 2001 From: Matthias Beyer Date: Tue, 6 Nov 2018 19:45:50 +0100 Subject: [PATCH] Remove call to format_err!(), use err_msg() instead Signed-off-by: Matthias Beyer --- lib/core/libimagrt/src/logger.rs | 25 ++++++++++--------------- 1 file changed, 10 insertions(+), 15 deletions(-) diff --git a/lib/core/libimagrt/src/logger.rs b/lib/core/libimagrt/src/logger.rs index 4d493a2e..de03b5d5 100644 --- a/lib/core/libimagrt/src/logger.rs +++ b/lib/core/libimagrt/src/logger.rs @@ -267,9 +267,8 @@ fn translate_destinations(raw: &Vec) -> Result> { acc.and_then(|mut v| { let dest = val.as_str() .ok_or_else(|| { - let path = "imag.logging.modules..destinations"; - let ty = "Array"; - Error::from(format_err!("Type error at {}, expected {}", path, ty)) + let msg = "Type error at 'imag.logging.modules..destinations', expected Array"; + Error::from(err_msg(msg)) }) .and_then(translate_destination)?; v.push(dest); @@ -290,9 +289,8 @@ fn aggregate_global_destinations(matches: &ArgMatches, config: Option<&Value>) .ok_or_else(|| err_msg("Global log destination config missing"))? .as_array() .ok_or_else(|| { - let path = "imag.logging.destinations"; - let ty = "Array"; - Error::from(format_err!("Type error at {}, expected {}", path, ty)) + let msg = "Type error at 'imag.logging.destinations', expected 'Array'"; + Error::from(err_msg(msg)) }) .and_then(translate_destinations), None => { @@ -393,9 +391,8 @@ fn aggregate_module_settings(_matches: &ArgMatches, config: Option<&Value>) .map(|val| { val.as_array() .ok_or_else(|| { - let path = "imag.logging.modules..destinations"; - let ty = "Array"; - Error::from(format_err!("Type error at {}, expected {}", path, ty)) + let msg = "Type error at 'imag.logging.modules..destinations', expected 'Array'"; + Error::from(err_msg(msg)) }) .and_then(translate_destinations) }) @@ -413,9 +410,8 @@ fn aggregate_module_settings(_matches: &ArgMatches, config: Option<&Value>) .context(EM::TomlQueryError)? .map(|v| v.as_bool().unwrap_or(false)) .ok_or_else(|| { - let path = "imag.logging.modules..enabled"; - let ty = "Boolean"; - Error::from(format_err!("Type error at {}, expected {}", path, ty)) + let msg = "Type error at 'imag.logging.modules..enabled', expected 'Boolean'"; + Error::from(err_msg(msg)) })?; let module_settings = ModuleSettings { @@ -431,9 +427,8 @@ fn aggregate_module_settings(_matches: &ArgMatches, config: Option<&Value>) Ok(settings) }, Ok(Some(_)) => { - let path = "imag.logging.modules"; - let ty = "Table"; - Err(Error::from(format_err!("Type error at {}, expected {}", path, ty))) + let msg = "Type error at 'imag.logging.modules', expected 'Table'"; + Err(Error::from(err_msg(msg))) }, Ok(None) => { // No modules configured. This is okay!