[PATCHv1 1/6] git-p4: make closeStreams() idempotent

From: Luke Diamand <luke@diamand.org>
Date: Jan 29, 2020, 11:12 UTC
Message-ID: <20200129111246.12196-2-luke@diamand.org>
In-Reply-To: <20200129111246.12196-1-luke@diamand.org>

Ensure that we can safely call self.closeStreams() multiple times, and can also call it even if there is no git fast-import stream at all.

Signed-off-by: Luke Diamand <luke@diamand.org>
---
 git-p4.py | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/git-p4.py b/git-p4.py
index 40d9e7c594..23724defe8 100755
--- a/git-p4.py
+++ b/git-p4.py
@@ -3549,11 +3549,14 @@ def openStreams(self):
         self.gitError = self.importProcess.stderr
 
     def closeStreams(self):
+        if self.gitStream is None:
+            return
         self.gitStream.close()
         if self.importProcess.wait() != 0:
             die("fast-import failed: %s" % self.gitError.read())
         self.gitOutput.close()
         self.gitError.close()
+        self.gitStream = None
 
     def run(self, args):
         if self.importIntoRemotes:

-- 
2.20.1.390.gb5101f9297