diff options
| author | KyleMiles <krm504@nyu.edu> | 2023-01-05 17:29:14 -0500 |
|---|---|---|
| committer | KyleMiles <krm504@nyu.edu> | 2023-01-06 16:29:58 -0500 |
| commit | a154e45cce79b0c2264c1e1cd37a3d1bf5bc6154 (patch) | |
| tree | 11f436b720edfdf37819e9e83e7131d220687074 /rust/src/fileaccessor.rs | |
| parent | 52edc39a7081fd6662e14fbcd473adabbbc36c7a (diff) | |
Rust API: Lots and lots of clippy changes
Diffstat (limited to 'rust/src/fileaccessor.rs')
| -rw-r--r-- | rust/src/fileaccessor.rs | 27 |
1 files changed, 9 insertions, 18 deletions
diff --git a/rust/src/fileaccessor.rs b/rust/src/fileaccessor.rs index 68b9cf14..874e348d 100644 --- a/rust/src/fileaccessor.rs +++ b/rust/src/fileaccessor.rs @@ -35,10 +35,7 @@ impl<'a> FileAccessor<'a> { { let f = unsafe { &mut *(ctxt as *mut F) }; - match f.seek(SeekFrom::End(0)) { - Ok(len) => len, - Err(_) => 0, - } + f.seek(SeekFrom::End(0)).unwrap_or(0) } extern "C" fn cb_read<F>( @@ -53,14 +50,11 @@ impl<'a> FileAccessor<'a> { let f = unsafe { &mut *(ctxt as *mut F) }; let dest = unsafe { slice::from_raw_parts_mut(dest as *mut u8, len) }; - if !f.seek(SeekFrom::Start(offset)).is_ok() { + if f.seek(SeekFrom::Start(offset)).is_err() { debug!("Failed to seek to offset {:x}", offset); - return 0; - } - - match f.read(dest) { - Ok(len) => len, - Err(_) => 0, + 0 + } else { + f.read(dest).unwrap_or(0) } } @@ -76,13 +70,10 @@ impl<'a> FileAccessor<'a> { let f = unsafe { &mut *(ctxt as *mut F) }; let src = unsafe { slice::from_raw_parts(src as *const u8, len) }; - if !f.seek(SeekFrom::Start(offset)).is_ok() { - return 0; - } - - match f.write(src) { - Ok(len) => len, - Err(_) => 0, + if f.seek(SeekFrom::Start(offset)).is_err() { + 0 + } else { + f.write(src).unwrap_or(0) } } |
