FS backend: Safe allocation of new PathBuf object

::stf::fs::create_dir_all() takes any ref to `Path`, which is what we
have here, so we can leave out the allocation of a new PathBuf object
here.

Also remove the match by a `if let Some(_)`, which increases the
readability a bit.

Signed-off-by: Matthias Beyer <mail@beyermatthias.de>

1 files changed, 7 insertions, 8 deletions

diff --git a/lib/core/libimagstore/src/file_abstraction/fs.rs b/lib/core/libimagstore/src/file_abstraction/fs.rs
index 131edcba..189032ba 100644
--- a/lib/core/libimagstore/src/file_abstraction/fs.rs
+++ b/lib/core/libimagstore/src/file_abstraction/fs.rs
@@ -105,15 +105,14 @@ impl FileAbstraction for FSFileAbstraction {
     }
 
     fn rename(&self, from: &PathBuf, to: &PathBuf) -> Result<()> {
-        match to.parent() {
-            Some(p) => if !p.exists() {
+        if let Some(p) = to.parent() {
+            if !p.exists() {
                 debug!("Creating: {:?}", p);
-                let _ = create_dir_all(&PathBuf::from(p)).context(EM::DirNotCreated)?;
-            },
-            None => {
-                debug!("Failed to find parent. This looks like it will fail now");
-                //nothing
-            },
+                let _ = create_dir_all(&p).context(EM::DirNotCreated)?;
+            }
+        } else {
+            debug!("Failed to find parent. This looks like it will fail now");
+            //nothing
         }
 
         debug!("Renaming {:?} to {:?}", from, to);