Cleanup pylint nitpicks

4 files changed, 26 insertions, 30 deletions

diff --git a/pydora/configure.py b/pydora/configure.py
index 21ea3c3..824ab03 100644
--- a/pydora/configure.py
+++ b/pydora/configure.py
@@ -10,7 +10,7 @@ from pandora.clientbuilder import PydoraConfigFileBuilder
 from .utils import Screen, Colors
 
 
-class umask(object):
+class Umask(object):
     """Set/Restore Umask Context Manager
     """
 
@@ -36,7 +36,7 @@ class PandoraKeysConfigParser(object):
                 "plain/json/partners.rst")
 
     FIELD_RE = re.compile(
-            ":(?P<key>[^:]+): (?:`{2})?(?P<value>[^`\n]+)(?:`{2})?$")
+        ":(?P<key>[^:]+): (?:`{2})?(?P<value>[^`\n]+)(?:`{2})?$")
 
     def _fixup_key(self, key):
         key = key.lower()
@@ -89,8 +89,8 @@ class PandoraKeysConfigParser(object):
             elif self._is_device_terminator(line):
                 key = self._clean_device_name(buffer.pop())
                 current_partner = partners[key] = {
-                        "api_host": self._format_api_host(api_host)
-                        }
+                    "api_host": self._format_api_host(api_host)
+                    }
 
             buffer.append(line.strip().lower())
 
@@ -142,8 +142,8 @@ class Configurator(object):
             self.cfg.set("api", key, value)
 
     def write_config(self):
-        with umask(0o077), open(self.builder.path, "w") as fp:
-            self.cfg.write(fp)
+        with Umask(0o077), open(self.builder.path, "w") as file:
+            self.cfg.write(file)
 
     def configure(self):
         if self.builder.file_exists:
diff --git a/pydora/mpg123.py b/pydora/mpg123.py
index e60ab43..796a8ab 100644
--- a/pydora/mpg123.py
+++ b/pydora/mpg123.py
@@ -70,10 +70,10 @@ class Player(object):
                 readers, _, _ = select.select(
                     [self._control_channel, self._process.stdout], [], [], 1)
 
-                for fd in readers:
-                    value = fd.readline().strip()
+                for handle in readers:
+                    value = handle.readline().strip()
 
-                    if fd.fileno() == self._control_fd:
+                    if handle.fileno() == self._control_fd:
                         self._callbacks.input(value, song)
                     else:
                         if self._player_stopped(value):
diff --git a/pydora/player.py b/pydora/player.py
index 4561442..6e93df5 100755
--- a/pydora/player.py
+++ b/pydora/player.py
@@ -10,7 +10,7 @@ from __future__ import print_function
 
 import os
 import sys
-from pandora import APIClient, clientbuilder
+from pandora import clientbuilder
 
 from .mpg123 import Player
 from .utils import Colors, Screen
@@ -82,9 +82,9 @@ class PlayerApp(object):
         """
         Screen.clear()
 
-        for i, s in enumerate(self.stations):
+        for i, station in enumerate(self.stations):
             i = "{:>3}".format(i)
-            print("{}: {}".format(Colors.yellow(i), s.name))
+            print("{}: {}".format(Colors.yellow(i), station.name))
 
         return self.stations[Screen.get_integer("Station: ")]
 
@@ -95,7 +95,7 @@ class PlayerApp(object):
             print("{} ".format(Colors.cyan("Advertisement")))
         else:
             print("{} by {}".format(Colors.cyan(song.song_name),
-                  Colors.yellow(song.artist_name)))
+                                    Colors.yellow(song.artist_name)))
 
     def skip_song(self, song):
         if song.is_ad:
diff --git a/pydora/utils.py b/pydora/utils.py
index 275c386..3c3aa39 100644
--- a/pydora/utils.py
+++ b/pydora/utils.py
@@ -5,8 +5,6 @@ import sys
 import termios
 import getpass
 import subprocess
-from pandora.models.pandora import AdItem
-from pandora import errors
 
 
 def input(prompt):
@@ -19,13 +17,13 @@ def input(prompt):
 
 class Colors(object):
 
-    def __wrap_with(code):
+    def __wrap_with(raw_code):
         @staticmethod
         def inner(text, bold=False):
-            c = code
+            code = raw_code
             if bold:
-                c = "1;{}".format(c)
-            return "\033[{}m{}\033[0m".format(c, text)
+                code = "1;{}".format(code)
+            return "\033[{}m{}\033[0m".format(code, text)
         return inner
 
     red = __wrap_with("31")
@@ -41,20 +39,18 @@ class Screen(object):
 
     @staticmethod
     def set_echo(enabled):
-        fd = sys.stdin.fileno()
-        if not os.isatty(fd):
+        handle = sys.stdin.fileno()
+        if not os.isatty(handle):
             return
 
-        (iflag, oflag, cflag,
-         lflag, ispeed, ospeed, cc) = termios.tcgetattr(fd)
+        attrs = termios.tcgetattr(handle)
 
         if enabled:
-            lflag |= termios.ECHO
+            attrs[3] |= termios.ECHO
         else:
-            lflag &= ~termios.ECHO
+            attrs[3] &= ~termios.ECHO
 
-        termios.tcsetattr(fd, termios.TCSANOW,
-                          [iflag, oflag, cflag, lflag, ispeed, ospeed, cc])
+        termios.tcsetattr(handle, termios.TCSANOW, attrs)
 
     @staticmethod
     def clear():
@@ -120,9 +116,9 @@ def iterate_forever(func, *args, **kwargs):
 
     while True:
         try:
-            playlistItem = next(output)
-            playlistItem.prepare_playback()
-            yield playlistItem
+            playlist_item = next(output)
+            playlist_item.prepare_playback()
+            yield playlist_item
         except StopIteration:
             output = func(*args, **kwargs)