Fixed all applicable lint issues

Author: Byron

gitdb/db/base.py

@@ -22,7 +22,6 @@
 from itertools import chain
 from functools import reduce
 
-import sys
 
 
 __all__ = ('ObjectDBR', 'ObjectDBW', 'FileDBBase', 'CompoundDB', 'CachingDB')
@@ -206,7 +205,6 @@ def _databases_recursive(database, output):
     """Fill output list with database from db, in order. Deals with Loose, Packed
     and compound databases."""
     if isinstance(database, CompoundDB):
-        compounds = list()
         dbs = database.databases()
         output.extend(db for db in dbs if not isinstance(db, CompoundDB))
         for cdb in (db for db in dbs if isinstance(db, CompoundDB)):

gitdb/db/git.py

@@ -12,12 +12,7 @@
 from gitdb.db.pack import PackedDB
 from gitdb.db.ref import ReferenceDB
 
-from gitdb.util import LazyMixin
-from gitdb.exc import (
-    InvalidDBRoot,
-    BadObject,
-    AmbiguousObjectName
-)
+from gitdb.exc import InvalidDBRoot
 
 import os
 

gitdb/db/loose.py

@@ -10,7 +10,6 @@
 
 
 from gitdb.exc import (
-    InvalidDBRoot,
     BadObject,
     AmbiguousObjectName
 )
@@ -55,8 +54,6 @@
 from gitdb.utils.encoding import force_bytes
 
 import tempfile
-import mmap
-import sys
 import os
 
 
@@ -149,11 +146,6 @@ def _map_loose_object(self, sha):
                 raise BadObject(sha)
             # END handle error
         # END exception handling
-        try:
-            return mmap.mmap(fd, 0, access=mmap.ACCESS_READ)
-        finally:
-            os.close(fd)
-        # END assure file is closed
 
     def set_ostream(self, stream):
         """:raise TypeError: if the stream does not support the Sha1Writer interface"""

gitdb/db/pack.py

@@ -103,7 +103,6 @@ def stream(self, sha):
         return entity.stream_at_index(index)
 
     def sha_iter(self):
-        sha_list = list()
         for entity in self.entities():
             index = entity.index()
             sha_by_index = index.sha

gitdb/db/ref.py

@@ -6,7 +6,6 @@
     CompoundDB,
 )
 
-import os
 __all__ = ('ReferenceDB', )
 
 class ReferenceDB(CompoundDB):

gitdb/ext/smmap

@@ -6,18 +6,15 @@
 Keeping this code separate from the beginning makes it easier to out-source
 it into c later, if required"""
 
-from gitdb.exc import (
-    BadObjectType
-)
-
 import zlib
 from gitdb.util import byte_ord
 decompressobj = zlib.decompressobj
 
 import mmap
 from itertools import islice
+from functools import reduce
 
-from gitdb.utils.compat import izip
+from gitdb.utils.compat import izip, buffer, xrange
 from gitdb.typ import (
     str_blob_type,
     str_commit_type,
@@ -247,7 +244,6 @@ def compress(self):
         if slen < 2:
             return self
         i = 0
-        slen_orig = slen
 
         first_data_index = None
         while i < slen:
@@ -399,7 +395,6 @@ def loose_object_header_info(m):
         object as well as its uncompressed size in bytes.
     :param m: memory map from which to read the compressed object data"""
     from gitdb.const import NULL_BYTE
-    from gitdb.utils.encoding import force_text
 
     decompress_size = 8192      # is used in cgit as well
     hdr = decompressobj().decompress(m, decompress_size)
@@ -684,7 +679,7 @@ def is_equal_canonical_sha(canonical_length, match, sha1):
 
 
 try:
-    # raise ImportError; # DEBUG
+    # NOQA
     from _perf import connect_deltas
 except ImportError:
     pass

gitdb/fun.py

@@ -7,6 +7,7 @@
 
 from gitdb.exc import (
     BadObject,
+    AmbiguousObjectName,
     UnsupportedOperation,
     ParseError
 )
@@ -57,11 +58,7 @@
     FlexibleSha1Writer
 )
 
-from struct import (
-    pack,
-    unpack,
-)
-
+from struct import pack
 from binascii import crc32
 
 from gitdb.const import NULL_BYTE

gitdb/pack.py

@@ -3,9 +3,8 @@
 # This module is part of GitDB and is released under
 # the New BSD License: http://www.opensource.org/licenses/bsd-license.php
 
-from io import BytesIO, StringIO
+from io import BytesIO
 
-import errno
 import mmap
 import os
 import zlib
@@ -15,7 +14,6 @@
     stream_copy,
     apply_delta_data,
     connect_deltas,
-    DeltaChunkList,
     delta_types
 )
 

gitdb/stream.py

@@ -6,12 +6,16 @@
 from gitdb.test.lib import (
     with_rw_directory,
     with_packs_rw,
-    ZippedStoreShaWriter,
     fixture_path,
     TestBase
 )
 
-from gitdb.stream import Sha1Writer
+
+
+from gitdb.stream import (
+    Sha1Writer,
+    ZippedStoreShaWriter
+)
 
 from gitdb.base import (
     IStream,

gitdb/test/db/lib.py

@@ -2,7 +2,11 @@
 #
 # This module is part of GitDB and is released under
 # the New BSD License: http://www.opensource.org/licenses/bsd-license.php
-from gitdb.test.db.lib import *
+from gitdb.test.db.lib import (
+    TestDBBase, 
+    fixture_path,
+    with_rw_directory
+)
 from gitdb.exc import BadObject
 from gitdb.db import GitDB
 from gitdb.base import OStream, OInfo

gitdb/test/db/test_git.py

@@ -2,7 +2,10 @@
 #
 # This module is part of GitDB and is released under
 # the New BSD License: http://www.opensource.org/licenses/bsd-license.php
-from gitdb.test.db.lib import *
+from gitdb.test.db.lib import (
+    TestDBBase, 
+    with_rw_directory
+)
 from gitdb.db import LooseObjectDB
 from gitdb.exc import BadObject
 from gitdb.util import bin_to_hex

gitdb/test/db/test_loose.py

@@ -2,7 +2,10 @@
 #
 # This module is part of GitDB and is released under
 # the New BSD License: http://www.opensource.org/licenses/bsd-license.php
-from gitdb.test.db.lib import *
+from gitdb.test.db.lib import (
+    TestDBBase,
+    with_rw_directory
+)
 from gitdb.db import (
     MemoryDB,
     LooseObjectDB

gitdb/test/db/test_mem.py

@@ -2,9 +2,12 @@
 #
 # This module is part of GitDB and is released under
 # the New BSD License: http://www.opensource.org/licenses/bsd-license.php
-from gitdb.test.db.lib import *
+from gitdb.test.db.lib import (
+    TestDBBase,
+    with_rw_directory,
+    with_packs_rw
+)
 from gitdb.db import PackedDB
-from gitdb.test.lib import fixture_path
 
 from gitdb.exc import BadObject, AmbiguousObjectName
 
@@ -46,8 +49,8 @@ def test_writing(self, path):
         random.shuffle(sha_list)
 
         for sha in sha_list:
-            info = pdb.info(sha)
-            stream = pdb.stream(sha)
+            pdb.info(sha)
+            pdb.stream(sha)
         # END for each sha to query
 
 

gitdb/test/db/test_pack.py

@@ -3,14 +3,7 @@
 # This module is part of GitDB and is released under
 # the New BSD License: http://www.opensource.org/licenses/bsd-license.php
 """Utilities used in ODB testing"""
-from gitdb import (
-    OStream,
-    )
-from gitdb.stream import (
-    Sha1Writer,
-    ZippedStoreShaWriter
-)
-
+from gitdb import OStream
 from gitdb.utils.compat import xrange
 
 import sys

gitdb/test/lib.py

@@ -4,9 +4,7 @@
 # the New BSD License: http://www.opensource.org/licenses/bsd-license.php
 """Contains library functions"""
 import os
-from gitdb.test.lib import *
-import shutil
-import tempfile
+from gitdb.test.lib import TestBase
 
 
 #{ Invvariants

gitdb/test/performance/lib.py

@@ -9,11 +9,11 @@
 
 from gitdb.exc import UnsupportedOperation
 from gitdb.db.pack import PackedDB
+from gitdb.utils.compat import xrange
 
 import sys
 import os
 from time import time
-import random
 
 from nose import SkipTest
 

gitdb/test/performance/test_pack.py

@@ -38,7 +38,6 @@ def test_pack_writing(self):
 
         ni = 5000
         count = 0
-        total_size = 0
         st = time()
         for sha in pdb.sha_iter():
             count += 1

gitdb/test/performance/test_pack_streaming.py

@@ -4,13 +4,13 @@
 # the New BSD License: http://www.opensource.org/licenses/bsd-license.php
 """Performance data streaming performance"""
 from lib import TestBigRepoR
-from gitdb.db import *
-from gitdb.base import *
-from gitdb.stream import *
+from gitdb.db import LooseObjectDB
+from gitdb.stream import IStream
+
 from gitdb.util import (
-                            pool,
-                            bin_to_hex
-                        )
+    pool,
+    bin_to_hex
+)
 from gitdb.typ import str_blob_type
 from gitdb.fun import chunk_size
 
@@ -19,19 +19,15 @@
     ChannelThreadTask,
     )
 
-from cStringIO import StringIO
 from time import time
 import os
 import sys
-import stat
-import subprocess
 
 
 from lib import (
-    TestBigRepoR,
     make_memory_file,
     with_rw_directory
-    )
+)
 
 
 #{ Utilities

gitdb/test/performance/test_stream.py

@@ -9,7 +9,15 @@
         DeriveTest,
     )
 
-from gitdb import *
+from gitdb import (
+    OInfo,
+    OPackInfo,
+    ODeltaPackInfo,
+    OStream,
+    OPackStream,
+    ODeltaPackStream,
+    IStream
+)
 from gitdb.util import (
     NULL_BIN_SHA
     )

gitdb/test/test_base.py

@@ -3,7 +3,10 @@
 # This module is part of GitDB and is released under
 # the New BSD License: http://www.opensource.org/licenses/bsd-license.php
 """Module with examples from the tutorial section of the docs"""
-from gitdb.test.lib import *
+from gitdb.test.lib import (
+    TestBase, 
+    fixture_path
+)
 from gitdb import IStream
 from gitdb.db import LooseObjectDB
 from gitdb.util import pool

gitdb/test/test_example.py

@@ -6,7 +6,6 @@
 from gitdb.test.lib import (
     TestBase,
     with_rw_directory,
-    with_packs_rw,
     fixture_path
 )
 
@@ -27,7 +26,6 @@
 from gitdb.exc import UnsupportedOperation
 from gitdb.util import to_bin_sha
 from gitdb.utils.compat import xrange
-from itertools import chain
 
 try:
     from itertools import izip
@@ -37,7 +35,6 @@
 from nose import SkipTest
 
 import os
-import sys
 import tempfile