matthiasbeyer/imag
1
0
Fork 0
Code Releases Activity

Merge pull request #746 from matthiasbeyer/libimagstorestdhook/git-refactor-error-mapping

Browse source 
libimagstorestdhook/git: refactor error mapping
This commit is contained in:
Matthias Beyer 2016-09-18 18:08:52 +02:00 committed by GitHub
commit 614b90e643
2 changed files with 8 additions and 19 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

16
libimagstore/src/store.rs
View file

@ -493,10 +493,7 @@ impl Store {
/// Return the `FileLockEntry` and write to disk
pub fn update<'a>(&'a self, mut entry: FileLockEntry<'a>) -> Result<()> {
if let Err(e) = self._update(&mut entry) {
return Err(e).map_err_into(SEK::UpdateCallError);
}
Ok(())
self._update(&mut entry).map_err_into(SEK::UpdateCallError)
}
/// Internal method to write to the filesystem store.
@ -505,12 +502,11 @@ impl Store {
/// This method assumes that entry is dropped _right after_ the call, hence
/// it is not public.
fn _update<'a>(&'a self, entry: &mut FileLockEntry<'a>) -> Result<()> {
if let Err(e) = self.execute_hooks_for_mut_file(self.pre_update_aspects.clone(), entry) {
return Err(e)
.map_err_into(SEK::PreHookExecuteError)
.map_err_into(SEK::HookExecutionError)
.map_err_into(SEK::UpdateCallError);
}
let _ = try!(self.execute_hooks_for_mut_file(self.pre_update_aspects.clone(), entry)
.map_err_into(SEK::PreHookExecuteError)
.map_err_into(SEK::HookExecutionError)
.map_err_into(SEK::UpdateCallError)
);
let mut hsmap = match self.entries.write() {
Err(_) => return Err(SE::new(SEK::LockPoisoned, None)),

11
libimagstorestdhook/src/vcs/git/runtime.rs
View file

@ -4,7 +4,7 @@ use git2::{Index, Repository};
use toml::Value;
use libimagerror::into::IntoError;
use libimagerror::trace::trace_error;
use libimagerror::trace::{MapErrTrace, trace_error};
use libimagstore::hook::error::CustomData;
use libimagstore::hook::error::HookErrorKind as HEK;
use libimagstore::hook::result::HookResult;
@ -31,14 +31,7 @@ impl Runtime {
/// returns a `Runtime` object that does _not_ contain a `Repository`.
pub fn new(storepath: &PathBuf) -> Runtime {
Runtime {
repository: match Repository::open(storepath) {
Ok(r) => Some(r),
Err(e) => {
trace_error(&e);
None
},
},
repository: Repository::open(storepath).map_err_trace().ok(),
config: None,
}
}