Code quality

Author: PeterSurda

src/bitmessageqt/__init__.py

@@ -1921,7 +1921,7 @@ def updateSentItemStatusByToAddress(self, toAddress, textToDisplay):
                         sent.item(i, 3).setText(textToDisplay)
 
     def updateSentItemStatusByAckdata(self, ackdata, textToDisplay):
-        if type(ackdata) is str:
+        if isinstance(ackdata, str):
             ackdata = QtCore.QByteArray(ackdata)
         for sent in (
             self.ui.tableWidgetInbox,
@@ -3193,7 +3193,7 @@ def deleteRowFromMessagelist(
                 self.ui.tableWidgetInboxChans,
                 self.ui.tableWidgetInboxSubscriptions
             )
-        elif type(messageLists) not in (list, tuple):
+        elif not isinstance(messageLists, (list, tuple)):
             messageLists = (messageLists,)
         for messageList in messageLists:
             if row is not None:
@@ -4144,7 +4144,7 @@ def tableWidgetInboxItemClicked(self):
             message = _translate(
                 "MainWindow",
                 "Error occurred: could not load message from disk."
-            )
+            )  # pylint: disable=redefined-variable-type
         else:
             tableWidget = self.getCurrentMessagelist()
             currentRow = tableWidget.currentRow()
@@ -4168,9 +4168,10 @@ def tableWidgetAddressBookItemChanged(self, item):
         self.rerenderMessagelistFromLabels()
         self.rerenderMessagelistToLabels()
         completerList = self.ui.lineEditTo.completer().model().stringList()
-        for i in range(len(completerList)):
-            if text_type(completerList[i]).endswith(" <" + item.address + ">"):
-                completerList[i] = item.label + " <" + item.address + ">"
+        for index_, string_ in enumerate(completerList):
+            if text_type(string_).endswith(" <" + item.address + ">"):
+                completerList[index_] = item.label + " <" \
+                    + item.address + ">"
         self.ui.lineEditTo.completer().model().setStringList(completerList)
 
     def tabWidgetCurrentChanged(self, n):

src/network/asyncore_pollchoose.py

@@ -478,7 +478,7 @@ def loop(timeout=30.0, use_poll=False, map=None, count=None, poller=None):
         # then poll
         poller(subtimeout, map)
         if isinstance(count, int):
-            count = count - 1
+            count = count - 1  # pylint: disable=redefined-variable-type
 
 
 class dispatcher(object):

src/tests/test_process.py

@@ -74,6 +74,7 @@ def setUpClass(cls):
 
         try:
             pid = int(cls._get_readline('singleton.lock'))
+            # pylint: disable=redefined-variable-type
             cls.process = psutil.Process(pid)
             time.sleep(5)
         except (psutil.NoSuchProcess, TypeError):