Remove unnecessary else after raise/return

holland-backup · Mar 19, 2024 · 3ac52ea · 3ac52ea
1 parent a4b701b
commit 3ac52ea
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 6 deletions.
diff --git a/contrib/holland-commvault/holland_commvault/commvault.py b/contrib/holland-commvault/holland_commvault/commvault.py
@@ -156,8 +156,7 @@ def main():
         except IOError:
             logging.info("Holland (commvault agent) failed to open/read status file")
             return 1
-        else:
-            status.close()
+        status.close()
         return ret
     except IOError as ex:
         logging.warning("Holland (commvault agent) IOError: %r", ex)

diff --git a/holland/core/util/lock.py b/holland/core/util/lock.py
@@ -31,8 +31,7 @@ def acquire(self):
         except IOError as exc:
             self.lock = None
             raise LockError(str(exc), exc)
-        else:
-            return True
+        return True
 
     def is_locked(self):
         """Check for lock"""
@@ -48,5 +47,4 @@ def release(self):
             self.lock = None
         except IOError as exc:
             raise LockError(str(exc), exc)
-        else:
-            return True
+        return True