matthiasbeyer
/
imag
1
0
Fork 0
Code Releases Activity

Use ok_or_else() instead of ok_or()

This commit is contained in:
Matthias Beyer 2018-04-24 21:16:55 +02:00
parent 22170a0d55
commit f1142c414d
4 changed files with 8 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
lib/core/libimagstore/src/configuration.rs
View File

@ -31,7 +31,7 @@ pub fn config_implicit_store_create_allowed(config: &Option<Value>) -> Result<bo
let key = "store.implicit-create";
if let Some(ref t) = *config {
t.read_bool(key)?.ok_or(SE::from_kind(SEK::ConfigKeyMissingError(key)))
t.read_bool(key)?.ok_or_else(|| SE::from_kind(SEK::ConfigKeyMissingError(key)))
} else {
Ok(false)
}

8
lib/core/libimagstore/src/file_abstraction/inmemory.rs
View File

@ -71,7 +71,7 @@ impl FileAbstractionInstance for InMemoryFileAbstractionInstance {
mtx.get_mut()
.get(&self.absent_path)
.cloned()
.ok_or(SE::from_kind(SEK::FileNotFound))
.ok_or_else(|| SE::from_kind(SEK::FileNotFound))
})
}
@ -123,7 +123,7 @@ impl FileAbstraction for InMemoryFileAbstraction {
.get_mut()
.remove(path)
.map(|_| ())
.ok_or(SE::from_kind(SEK::FileNotFound))
.ok_or_else(|| SE::from_kind(SEK::FileNotFound))
}
fn copy(&self, from: &PathBuf, to: &PathBuf) -> Result<(), SE> {
@ -131,7 +131,7 @@ impl FileAbstraction for InMemoryFileAbstraction {
let mut mtx = self.backend().lock().expect("Locking Mutex failed");
let backend = mtx.get_mut();
let a = backend.get(from).cloned().ok_or(SE::from_kind(SEK::FileNotFound))?;
let a = backend.get(from).cloned().ok_or_else(|| SE::from_kind(SEK::FileNotFound))?;
backend.insert(to.clone(), a);
debug!("Copying: {:?} -> {:?} worked", from, to);
Ok(())
@ -142,7 +142,7 @@ impl FileAbstraction for InMemoryFileAbstraction {
let mut mtx = self.backend().lock().expect("Locking Mutex failed");
let backend = mtx.get_mut();
let a = backend.get(from).cloned().ok_or(SE::from_kind(SEK::FileNotFound))?;
let a = backend.get(from).cloned().ok_or_else(|| SE::from_kind(SEK::FileNotFound))?;
backend.insert(to.clone(), a);
debug!("Renaming: {:?} -> {:?} worked", from, to);
Ok(())

4
lib/core/libimagstore/src/store.rs
View File

@ -1007,13 +1007,13 @@ fn has_only_tables(t: &Value) -> Result<bool> {
fn has_main_section(t: &Value) -> Result<bool> {
t.read("imag")?
.ok_or(SE::from_kind(SEK::ConfigKeyMissingError("imag")))
.ok_or_else(|| SE::from_kind(SEK::ConfigKeyMissingError("imag")))
.map(Value::is_table)
}
fn has_imag_version_in_main_section(t: &Value) -> Result<bool> {
t.read_string("imag.version")?
.ok_or(SE::from_kind(SEK::ConfigKeyMissingError("imag.version")))
.ok_or_else(|| SE::from_kind(SEK::ConfigKeyMissingError("imag.version")))
.map(String::from)
.map(|s| ::semver::Version::parse(&s).is_ok())
}

2
lib/core/libimagstore/src/storeid.rs
View File

@ -116,7 +116,7 @@ impl StoreId {
.unwrap_or_else(|| self.id.clone())
.to_str()
.map(String::from)
.ok_or(SE::from_kind(SEK::StoreIdHandlingError))
.ok_or_else(|| SE::from_kind(SEK::StoreIdHandlingError))
}
/// Returns the components of the `id` part of the StoreId object.