Code quality

Author: PeterSurda

src/bitmessageqt/bitmessageui.py

@@ -7,7 +7,6 @@
 #
 # WARNING! All changes made in this file will be lost!
 
-# pylint: disable=attribute-defined-outside-init
 from PyQt4 import QtCore, QtGui  # pylint: disable=import-error
 from bmconfigparser import config
 from foldertree import AddressBookCompleter
@@ -57,6 +56,7 @@ def _translate(context, text, disambig,
 
 
 class Ui_MainWindow(object):
+    # pylint: disable=attribute-defined-outside-init
     def setupUi(self, MainWindow):
         MainWindow.setObjectName(_fromUtf8("MainWindow"))
         MainWindow.resize(885, 580)
@@ -182,7 +182,8 @@ def setupUi(self, MainWindow):
         self.horizontalSplitter_3.setCollapsible(1, False)
         self.gridLayout.addWidget(self.horizontalSplitter_3)
         icon2 = QtGui.QIcon()
-        icon2.addPixmap(QtGui.QPixmap(_fromUtf8(":/newPrefix/images/inbox.png")), QtGui.QIcon.Normal, QtGui.QIcon.Off)
+        icon2.addPixmap(QtGui.QPixmap(_fromUtf8(":/newPrefix/images/inbox.png")),
+                        QtGui.QIcon.Normal, QtGui.QIcon.Off)
         self.tabWidget.addTab(self.inbox, icon2, _fromUtf8(""))
         self.send = QtGui.QWidget()
         self.send.setObjectName(_fromUtf8("send"))
@@ -204,7 +205,8 @@ def setupUi(self, MainWindow):
         item = QtGui.QTableWidgetItem()
         icon3 = QtGui.QIcon()
         icon3.addPixmap(
-            QtGui.QPixmap(_fromUtf8(":/newPrefix/images/addressbook.png")), QtGui.QIcon.Selected, QtGui.QIcon.Off
+            QtGui.QPixmap(_fromUtf8(":/newPrefix/images/addressbook.png")),
+            QtGui.QIcon.Selected, QtGui.QIcon.Off
         )
         item.setIcon(icon3)
         self.tableWidgetAddressBook.setHorizontalHeaderItem(0, item)
@@ -366,7 +368,8 @@ def setupUi(self, MainWindow):
         self.labelHumanFriendlyTTLDescription.setSizePolicy(sizePolicy)
         self.labelHumanFriendlyTTLDescription.setMinimumSize(QtCore.QSize(45, 0))
         self.labelHumanFriendlyTTLDescription.setObjectName(_fromUtf8("labelHumanFriendlyTTLDescription"))
-        self.horizontalLayout_5.addWidget(self.labelHumanFriendlyTTLDescription, 1, QtCore.Qt.AlignLeft)
+        self.horizontalLayout_5.addWidget(self.labelHumanFriendlyTTLDescription,
+                                          1, QtCore.Qt.AlignLeft)
         self.pushButtonClear = QtGui.QPushButton(self.send)
         self.pushButtonClear.setObjectName(_fromUtf8("pushButtonClear"))
         self.horizontalLayout_5.addWidget(self.pushButtonClear, 0, QtCore.Qt.AlignRight)
@@ -388,7 +391,8 @@ def setupUi(self, MainWindow):
         self.horizontalSplitter.setCollapsible(1, False)
         self.gridLayout_7.addWidget(self.horizontalSplitter, 0, 0, 1, 1)
         icon4 = QtGui.QIcon()
-        icon4.addPixmap(QtGui.QPixmap(_fromUtf8(":/newPrefix/images/send.png")), QtGui.QIcon.Normal, QtGui.QIcon.Off)
+        icon4.addPixmap(QtGui.QPixmap(_fromUtf8(":/newPrefix/images/send.png")),
+                        QtGui.QIcon.Normal, QtGui.QIcon.Off)
         self.tabWidget.addTab(self.send, icon4, _fromUtf8(""))
         self.subscriptions = QtGui.QWidget()
         self.subscriptions.setObjectName(_fromUtf8("subscriptions"))
@@ -691,7 +695,6 @@ def setupUi(self, MainWindow):
         MainWindow.setTabOrder(self.textEditMessage, self.pushButtonAddSubscription)
 
         # Popup menu actions container for the Sent page
-        # pylint: disable=attribute-defined-outside-init
         self.sentContextMenuToolbar = QtGui.QToolBar()
         # Popup menu actions container for chans tree
         self.addressContextMenuToolbar = QtGui.QToolBar()
@@ -825,14 +828,3 @@ def retranslateUi(self, MainWindow):
         self.actionDeleteAllTrashedMessages.setText(_translate("MainWindow", "Delete all trashed messages", None))
         self.actionJoinChan.setText(_translate("MainWindow", "Join / Create chan", None))
         self.updateNetworkSwitchMenuLabel()
-
-
-if __name__ == "__main__":
-    import sys
-
-    app = QtGui.QApplication(sys.argv)
-    MainWindow = settingsmixin.SMainWindow()
-    ui = Ui_MainWindow()
-    ui.setupUi(MainWindow)
-    MainWindow.show()
-    sys.exit(app.exec_())

src/bitmessageqt/tests/test_widgets.py

@@ -8,6 +8,7 @@
 import sys
 import unittest
 
+# pylint: disable=import-outside-toplevel,unused-import
 try:
     from PyQt4 import QtCore, QtGui, QtTest
     has_qt = True

src/build_osx.py

@@ -1,7 +1,7 @@
 """Building osx."""
 import os
 from glob import glob
-from PyQt4 import QtCore
+from PyQt4 import QtCore  # pylint: disable=import-error
 from setuptools import setup
 
 name = "Bitmessage"

src/class_sqlThread.py

@@ -34,8 +34,12 @@ class sqlThread(threading.Thread):
 
     def __init__(self):
         threading.Thread.__init__(self, name="SQL")
+        self.conn = None
+        self.cur = None
 
-    def run(self):  # pylint: disable=too-many-locals, too-many-branches, too-many-statements
+    # pylint: disable=protected-access,too-many-locals
+    # pylint: disable=too-many-branches,too-many-statements
+    def run(self):
         """Process SQL queries from `.helper_sql.sqlSubmitQueue`"""
         helper_sql.sql_available = True
         config_ready.wait()

src/plugins/notification_notify2.py

@@ -5,7 +5,7 @@
 
 import gi
 gi.require_version('Notify', '0.7')
-from gi.repository import Notify  # noqa:E402 pylint: disable=import-error
+from gi.repository import Notify  # noqa:E402 pylint: disable=import-error,no-name-in-module
 
 Notify.init('pybitmessage')
 

src/pyelliptic/openssl.py

@@ -9,7 +9,7 @@
 import ctypes
 import sys
 
-# pylint: disable=protected-access
+# pylint: disable=no-member,protected-access
 
 OpenSSL = None