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 87da1c28d0ebde152b749af1a23b1b86cb391aae..6a646e7825871c7488cd068c92c96b2b69582d75 100755 (executable)
--- 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):