Apply black -l 79 and do some code cleanup

README.md

@@ -50,8 +50,8 @@ Lib Usage
     ```python
     archive = destream.open("some_file.tar.gz")
 
-    assert isinstance(archive, destream.ArchivePack) \
-        and isinstance(archive.tarfile, tarfile.TarFile)
+    assert (isinstance(archive, destream.ArchivePack) and
+           isinstance(archive.tarfile, tarfile.TarFile))
 
     # ==> we can extract members using extract() and extractall()
     archive.extractall("/tmp")

destream/archive.py

@@ -8,16 +8,18 @@
           """.split()
 
 
-RE_EXTENSION = re.compile(r'^(.*?)(\.([^.]+))?$')
+RE_EXTENSION = re.compile(r"^(.*?)(\.([^.]+))?$")
 
 
 class Archive(io.BufferedReader):
     """
     Base class to Archive file
     """
+
     def __init__(self, name, fileobj=None, source=None, closefd=True):
-        assert type(self) != Archive, \
-            "This class can not be used in standalone"
+        assert (
+            type(self) != Archive
+        ), "This class can not be used in standalone"
         if not fileobj:
             fileobj = io.BytesIO()
         elif sys.version_info < (3, 0) and isinstance(fileobj, file):
@@ -32,7 +34,7 @@ def __init__(self, name, fileobj=None, source=None, closefd=True):
             f"got {type(fileobj)}"
         )
         io.BufferedReader.__init__(self, fileobj)
-        self.realname = name or ''
+        self.realname = name or ""
         self.source = source
         self.closefd = closefd
         if isinstance(source, Archive):
@@ -41,43 +43,46 @@ def __init__(self, name, fileobj=None, source=None, closefd=True):
         else:
             self._decompressors = [type(self)]
             self.compressions = []
-        if hasattr(self, '_compression'):
+        if hasattr(self, "_compression"):
             self.compressions += [self._compression]
 
     @classmethod
-    def _check_availability(self):
+    def _check_availability(cls):
         pass
 
     @classmethod
     def _guess(cls, mime, name, fileobj):
-        if getattr(cls, '_unique_instance', False):
+        if getattr(cls, "_unique_instance", False):
             if cls in fileobj._decompressors:
                 raise ValueError(
                     f"class {cls} already in the decompressor list"
                 )
         realname = name
-        if hasattr(cls, '_mimes'):
+        if hasattr(cls, "_mimes"):
             match = RE_EXTENSION.search(name)
-            if hasattr(cls, '_extensions') and \
-               match.group(2) and \
-               os.path.normcase(match.group(3)) in cls._extensions:
+            if (
+                hasattr(cls, "_extensions")
+                and match.group(2)
+                and os.path.normcase(match.group(3)) in cls._extensions
+            ):
                 realname = match.group(1)
             if mime not in cls._mimes:
                 raise ValueError(
                     (cls, mime, name, fileobj),
-                    f"can not decompress fileobj using class {cls.__name__}"
+                    f"can not decompress fileobj using class {cls.__name__}",
                 )
         return realname
 
     def close(self):
-        if getattr(self, 'closefd', True):
+        if getattr(self, "closefd", True):
             super().close()
 
 
 class ArchivePack(Archive):
     """
     Base class for an archive that is also a pack of file (tar, zip, ...)
     """
+
     def __init__(self, name, fileobj=None, source=None):
         Archive.__init__(self, name, fileobj, source=source)
 
@@ -86,16 +91,20 @@ def single(self):
 
     def members(self):
         raise NotImplementedError(
-            f"class {type(self)} does not implement this method")
+            f"class {type(self)} does not implement this method"
+        )
 
     def open(self, member):
         raise NotImplementedError(
-            f"class {type(self)} does not implement this method")
+            f"class {type(self)} does not implement this method"
+        )
 
     def extract(self, member, path):
         raise NotImplementedError(
-            f"class {type(self)} does not implement this method")
+            f"class {type(self)} does not implement this method"
+        )
 
     def extractall(self, path, members=None):
         raise NotImplementedError(
-            f"class {type(self)} does not implement this method")
+            f"class {type(self)} does not implement this method"
+        )

destream/decompressors/__init__.py

@@ -15,7 +15,8 @@
 
 builtin_decompressors = [
     (0, symbol)
-    for name, symbol
-    in inspect.getmembers(sys.modules[__name__], inspect.isclass)
+    for name, symbol in inspect.getmembers(
+        sys.modules[__name__], inspect.isclass
+    )
     if issubclass(symbol, archive.Archive)
 ]

destream/decompressors/bzip2.py

@@ -1,16 +1,16 @@
 from destream import ExternalPipe
 
-__all__ = ['Bunzip2']
+__all__ = ["Bunzip2"]
 
 
 class Bunzip2(ExternalPipe):
-    _mimes = ['application/x-bzip2']
-    _extensions = ['bz2', 'bz', 'tbz2', 'tbz']
-    _command = ['bunzip2']
-    _compression = 'bzip2'
+    _mimes = ["application/x-bzip2"]
+    _extensions = ["bz2", "bz", "tbz2", "tbz"]
+    _command = ["bunzip2"]
+    _compression = "bzip2"
 
     @classmethod
     def _guess(cls, mime, name, fileobj):
-        is_tar = name.endswith('.tbz2') or name.endswith('.tbz')
+        is_tar = name.endswith(".tbz2") or name.endswith(".tbz")
         realname = super()._guess(mime, name, fileobj)
-        return realname + '.tar' if is_tar else realname
+        return realname + ".tar" if is_tar else realname

destream/decompressors/gzip.py

@@ -1,34 +1,34 @@
 from destream import ExternalPipe
 
-__all__ = ['Gunzip']
+__all__ = ["Gunzip"]
 
 
 class Gunzip(ExternalPipe):
     _mimes = [
-        'application/x-gzip',
-        'application/gzip',
+        "application/x-gzip",
+        "application/gzip",
     ]
-    _extensions = ['gz']
-    _command = ['gunzip']
-    _compression = 'gzip'
+    _extensions = ["gz"]
+    _command = ["gunzip"]
+    _compression = "gzip"
 
     @classmethod
     def _guess(cls, mime, name, fileobj):
         if mime not in cls._mimes:
             raise ValueError("not a gzip compression")
         lowered = name.lower()
-        if lowered.endswith('.gz'):
+        if lowered.endswith(".gz"):
             realname = name[:-3]
-        elif lowered.endswith('-gz'):
+        elif lowered.endswith("-gz"):
             realname = name[:-3]
-        elif lowered.endswith('.z'):
+        elif lowered.endswith(".z"):
             realname = name[:-2]
-        elif lowered.endswith('-z'):
+        elif lowered.endswith("-z"):
             realname = name[:-2]
-        elif lowered.endswith('_z'):
+        elif lowered.endswith("_z"):
             realname = name[:-2]
-        elif lowered.endswith('.tgz') or lowered.endswith('.taz'):
-            realname = name[:-4] + '.tar'
+        elif lowered.endswith(".tgz") or lowered.endswith(".taz"):
+            realname = name[:-4] + ".tar"
         else:
             realname = name
         return realname

destream/decompressors/lzma.py

@@ -1,10 +1,10 @@
 from destream import ExternalPipe
 
-__all__ = ['Unlzma']
+__all__ = ["Unlzma"]
 
 
 class Unlzma(ExternalPipe):
-    _mimes = ['application/x-lzma']
-    _extensions = ['lzma']
-    _command = 'unlzma -c'.split()
-    _compression = 'lzma'
+    _mimes = ["application/x-lzma"]
+    _extensions = ["lzma"]
+    _command = "unlzma -c".split()
+    _compression = "lzma"