Simplify code

It's clearer to use "if foo in bar" rather than catching KeyError.

1 files changed, 4 insertions, 4 deletions

diff --git a/obnamlib/plugins/fuse_plugin.py b/obnamlib/plugins/fuse_plugin.py
index 05d12395..e6800f20 100644
--- a/obnamlib/plugins/fuse_plugin.py
+++ b/obnamlib/plugins/fuse_plugin.py
@@ -215,9 +215,8 @@ class ObnamFuse(fuse.Fuse):
 
     def get_metadata(self, path):
         tracing.trace('path=%r', path)
-        try:
-            return self.metadatacache[path]
-        except KeyError:
+
+        if path not in self.metadatacache:
             if len(self.metadatacache) > self.MAX_METADATA_CACHE:
                 self.metadatacache.clear()
             metadata = self.obnam.repo.get_metadata(*self.get_gen_path(path))
@@ -227,7 +226,8 @@ class ObnamFuse(fuse.Fuse):
                 metadata.st_atime_sec = 0
             if metadata.st_mtime_sec < 0:
                 metadata.st_mtime_sec = 0
-            return metadata
+
+        return self.metadatacache[path]
 
     def get_stat(self, path):
         tracing.trace('path=%r', path)