matthiasbeyer/imag
1
0
Fork 0
Code Releases Activity

lazyfile.rs: Replace error boxing and creation by call to new helper function

Browse source
This commit is contained in:
Matthias Beyer 2016-06-27 18:00:12 +02:00
parent 0a05b9825c
commit 484520842a
1 changed files with 4 additions and 17 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

21
libimagstore/src/lazyfile.rs
View file

@ -1,6 +1,4 @@
use libimagerror::into::IntoError;
use error::{StoreError as SE, StoreErrorKind as SEK};
use error::{MapErrInto, StoreError as SE, StoreErrorKind as SEK};
use std::io::{Seek, SeekFrom};
use std::path::{Path, PathBuf};
use std::fs::{File, OpenOptions, create_dir_all};
@ -40,16 +38,10 @@ impl LazyFile {
// We seek to the beginning of the file since we expect each
// access to the file to be in a different context
f.seek(SeekFrom::Start(0))
.map_err(Box::new)
.map_err(|e| SEK::FileNotCreated.into_error_with_cause(e))
.map_err_into(SEK::FileNotCreated)
.map(|_| f)
},
LazyFile::Absent(ref p) => {
try!(open_file(p)
.map_err(Box::new)
.map_err(|e| SEK::FileNotFound.into_error_with_cause(e))
)
}
LazyFile::Absent(ref p) => try!(open_file(p).map_err_into(SEK::FileNotFound)),
};
*self = LazyFile::File(file);
if let LazyFile::File(ref mut f) = *self {
@ -65,12 +57,7 @@ impl LazyFile {
debug!("Creating lazy file: {:?}", self);
let file = match *self {
LazyFile::File(ref mut f) => return Ok(f),
LazyFile::Absent(ref p) => {
try!(create_file(p)
.map_err(Box::new)
.map_err(|e| SEK::FileNotFound.into_error_with_cause(e))
)
}
LazyFile::Absent(ref p) => try!(create_file(p).map_err_into(SEK::FileNotFound)),
};
*self = LazyFile::File(file);
if let LazyFile::File(ref mut f) = *self {