Use builtins.open rather than io.open

Author: rhpvorderman

src/isal/igzip.py

@@ -29,6 +29,7 @@
 Library to speed up its methods."""
 
 import argparse
+import builtins
 import gzip
 import io
 import os
@@ -342,9 +343,9 @@ def main():
         if args.file is None:
             in_file = sys.stdin.buffer
         else:
-            in_file = io.open(args.file, mode="rb")
+            in_file = builtins.open(args.file, mode="rb")
         if out_filepath is not None:
-            out_buffer = io.open(out_filepath, "wb")
+            out_buffer = builtins.open(out_filepath, "wb")
         else:
             out_buffer = sys.stdout.buffer
 
@@ -360,7 +361,7 @@ def main():
         else:
             in_file = IGzipFile(mode="rb", fileobj=sys.stdin.buffer)
         if out_filepath is not None:
-            out_file = io.open(out_filepath, mode="wb")
+            out_file = builtins.open(out_filepath, mode="wb")
         else:
             out_file = sys.stdout.buffer
 

src/isal/igzip_threaded.py

@@ -5,6 +5,7 @@
 # This file is part of python-isal which is distributed under the
 # PYTHON SOFTWARE FOUNDATION LICENSE VERSION 2.
 
+import builtins
 import io
 import multiprocessing
 import os
@@ -54,7 +55,7 @@ def open(filename, mode="rb", compresslevel=igzip._COMPRESS_LEVEL_TRADEOFF,
                 threads = 1
     open_mode = mode.replace("t", "b")
     if isinstance(filename, (str, bytes)) or hasattr(filename, "__fspath__"):
-        binary_file = io.open(filename, open_mode)
+        binary_file = builtins.open(filename, open_mode)
     elif hasattr(filename, "read") or hasattr(filename, "write"):
         binary_file = filename
     else:
@@ -261,6 +262,8 @@ def flush(self):
         self.raw.flush()
 
     def close(self) -> None:
+        if self._closed:
+            return
         self.flush()
         self.stop()
         if self.exception: