Code quality

Author: PeterSurda

src/depends.py

@@ -188,7 +188,7 @@ def try_import(module, log_extra=False):
             logger.error(
                 'On %s, try running "%s %s" as root.',
                 dist, PACKAGE_MANAGER[dist], PACKAGES[module][dist])
-        return False
+        return None
 
 
 def check_ripemd160():
@@ -274,6 +274,7 @@ def check_openssl():
     if sys.platform == 'win32':
         paths = ['libeay32.dll']
         if getattr(sys, 'frozen', False):
+            # pylint: disable=no-member
             paths.insert(0, os.path.join(sys._MEIPASS, 'libeay32.dll'))
     else:
         paths = ['libcrypto.so', 'libcrypto.so.1.0.0']

src/fallback/umsgpack/umsgpack.py

@@ -410,8 +410,6 @@ def _pack2(obj, fp, **options):
     >>> umsgpack.pack({u"compact": True, u"schema": 0}, f)
     >>>
     """
-    global compatibility
-
     ext_handlers = options.get("ext_handlers")
 
     if obj is None:
@@ -480,8 +478,6 @@ def _pack3(obj, fp, **options):
     >>> umsgpack.pack({u"compact": True, u"schema": 0}, f)
     >>>
     """
-    global compatibility
-
     ext_handlers = options.get("ext_handlers")
 
     if obj is None:
@@ -665,7 +661,6 @@ def _unpack_string(code, fp, options):
         raise Exception("logic error, not string: 0x%02x" % ord(code))
 
     # Always return raw bytes in compatibility mode
-    global compatibility
     if compatibility:
         return _read_except(fp, length)
 

src/network/bmproto.py

@@ -67,6 +67,25 @@ def __init__(self, address=None, sock=None):
         # track port check requests, only allow one per connection
         # completely disable port checks for now
         self.portCheckRequested = True
+        self.payloadLength = 0
+        self.payloadOffset = 0
+        self.invalid = False
+        self.timeOffset = 0
+        self.verackSent = False
+        self.verackReceived = False
+        self.payload = None
+        self.isSSL = False
+        self.remoteProtocolVersion = 0
+        self.nonce = 0
+        self.peerNode = None
+        self.streams = None
+        self.timestamp = 0
+        self.object = None
+        self.services = None
+        self.sockNode = None
+        self.magic = None
+        self.checksum = None
+        self.command = None
 
     def bm_proto_reset(self):
         """Reset the bitmessage object parser"""