matthiasbeyer/imag
1
0
Fork 0
Code Releases Activity

Remove unwrap() by matching, remove unneeded deref()

Browse source
This commit is contained in:
Matthias Beyer 2016-05-14 19:17:47 +02:00
parent 8d9bd5154d
commit 2a6e4c62fe
1 changed files with 7 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

16
libimagstore/src/store.rs
View file

@ -526,15 +526,13 @@ impl Store {
fle: &mut FileLockEntry)
-> Result<()>
{
let guard = aspects.deref().lock();
if guard.is_err() { return Err(SE::new(SEK::PreHookExecuteError, None)) }
guard.unwrap().deref().iter()
.fold(Ok(()), |acc, aspect| {
debug!("[Aspect][exec]: {:?}", aspect);
acc.and_then(|_| aspect.access_mut(fle))
})
.map_err(|e| SE::new(SEK::PreHookExecuteError, Some(Box::new(e))))
match aspects.lock() {
Err(_) => return Err(SE::new(SEK::PreHookExecuteError, None)),
Ok(g) => g
}.iter().fold(Ok(()), |acc, aspect| {
debug!("[Aspect][exec]: {:?}", aspect);
acc.and_then(|_| aspect.access_mut(fle))
}).map_err(|e| SE::new(SEK::PreHookExecuteError, Some(Box::new(e))))
}
}