Code quality

Author: PeterSurda

src/bitmessageqt/bitmessageui.py

@@ -25,17 +25,34 @@ def _fromUtf8(s):
 try:
     _encoding = QtGui.QApplication.UnicodeUTF8
 
-    def _translate(context, text, disambig, encoding=QtCore.QCoreApplication.CodecForTr, n=None):
+    # pylint: disable=unused-argument
+    def _translate(context, text, disambig,
+                   encoding=QtCore.QCoreApplication.CodecForTr, n=None):
         if n is None:
-            return QtGui.QApplication.translate(context, text, disambig, _encoding)
+            return QtGui.QApplication.translate(context,
+                                                text,
+                                                disambig,
+                                                _encoding)
         else:
-            return QtGui.QApplication.translate(context, text, disambig, _encoding, n)
+            return QtGui.QApplication.translate(context,
+                                                text,
+                                                disambig,
+                                                _encoding,
+                                                n)
 except AttributeError:
-    def _translate(context, text, disambig, encoding=QtCore.QCoreApplication.CodecForTr, n=None):
+    # pylint: disable=unused-argument
+    def _translate(context, text, disambig,
+                   encoding=QtCore.QCoreApplication.CodecForTr, n=None):
         if n is None:
-            return QtGui.QApplication.translate(context, text, disambig)
+            return QtGui.QApplication.translate(context,
+                                                text,
+                                                disambig)
         else:
-            return QtGui.QApplication.translate(context, text, disambig, QtCore.QCoreApplication.CodecForTr, n)
+            return QtGui.QApplication.translate(context,
+                                                text,
+                                                disambig,
+                                                QtCore.QCoreApplication.CodecForTr,
+                                                n)
 
 
 class Ui_MainWindow(object):
@@ -44,7 +61,8 @@ def setupUi(self, MainWindow):
         MainWindow.resize(885, 580)
         icon = QtGui.QIcon()
         icon.addPixmap(
-            QtGui.QPixmap(_fromUtf8(":/newPrefix/images/can-icon-24px.png")), QtGui.QIcon.Normal, QtGui.QIcon.Off
+            QtGui.QPixmap(_fromUtf8(":/newPrefix/images/can-icon-24px.png")),
+            QtGui.QIcon.Normal, QtGui.QIcon.Off
         )
         MainWindow.setWindowIcon(icon)
         MainWindow.setTabShape(QtGui.QTabWidget.Rounded)
@@ -53,7 +71,8 @@ def setupUi(self, MainWindow):
         self.gridLayout_10 = QtGui.QGridLayout(self.centralwidget)
         self.gridLayout_10.setObjectName(_fromUtf8("gridLayout_10"))
         self.tabWidget = QtGui.QTabWidget(self.centralwidget)
-        sizePolicy = QtGui.QSizePolicy(QtGui.QSizePolicy.Expanding, QtGui.QSizePolicy.Expanding)
+        sizePolicy = QtGui.QSizePolicy(QtGui.QSizePolicy.Expanding,
+                                       QtGui.QSizePolicy.Expanding)
         sizePolicy.setHorizontalStretch(0)
         sizePolicy.setVerticalStretch(0)
         sizePolicy.setHeightForWidth(self.tabWidget.sizePolicy().hasHeightForWidth())
@@ -78,7 +97,7 @@ def setupUi(self, MainWindow):
         self.verticalSplitter_12.setOrientation(QtCore.Qt.Vertical)
         self.treeWidgetYourIdentities = settingsmixin.STreeWidget(self.inbox)
         self.treeWidgetYourIdentities.setObjectName(_fromUtf8("treeWidgetYourIdentities"))
-        self.treeWidgetYourIdentities.resize(200, self.treeWidgetYourIdentities.height())
+        self.treeWidgetYourIdentities.resize(200,self.treeWidgetYourIdentities.height())
         icon1 = QtGui.QIcon()
         icon1.addPixmap(
             QtGui.QPixmap(_fromUtf8(":/newPrefix/images/identities.png")), QtGui.QIcon.Selected, QtGui.QIcon.Off

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"""