From: Johannes Berg <johannes@sipsolutions.net>
Date: Sun, 5 Dec 2021 20:19:52 +0000 (+0100)
Subject: NoOpContextManager: fix init/close/__del__ logic
X-Git-Url: https://arthur.barton.de/gitweb/?p=bup.git;a=commitdiff_plain;h=ffa1a813b41d59cb3bdcd96429b9cbb3535e2d88;hp=9478e2d88634bd22b3d0145b76926d719e457faa

NoOpContextManager: fix init/close/__del__ logic

Initialize self.closed, etc.

Signed-off-by: Johannes Berg <johannes@sipsolutions.net>
Reviewed-by: Rob Browning <rlb@defaultvalue.org>
[rlb@defaultvalue.org: split from larger commit; adjust commit message]
Signed-off-by: Rob Browning <rlb@defaultvalue.org>
Tested-by: Rob Browning <rlb@defaultvalue.org>
---

diff --git a/lib/bup/cmd/split.py b/lib/bup/cmd/split.py
index 87da1c2..6a646e7 100755
--- a/lib/bup/cmd/split.py
+++ b/lib/bup/cmd/split.py
@@ -44,13 +44,13 @@ bwlimit=   maximum bytes/sec to transmit to server
 
 class NoOpPackWriter:
     def __init__(self):
-        pass
+        self.closed = False
     def __enter__(self):
         return self
     def __exit__(self, type, value, traceback):
-        return None  # since close() does nothing
+        self.close()
     def close(self):
-        assert self.closed
+        self.closed = True
     def __del__(self):
         assert self.closed
     def new_blob(self, content):