From a693993905b66a63601cf5765da1788829b0f798 Mon Sep 17 00:00:00 2001
From: Stein Magnus Jodal <stein.magnus@jodal.no>
Date: Sat, 7 Feb 2015 17:09:33 +0100
Subject: [PATCH] flake8: Fix new warnings after flake8 upgrade

---
 mopidy/audio/actor.py       | 4 ++--
 mopidy/audio/playlists.py   | 2 +-
 mopidy/local/__init__.py    | 2 +-
 mopidy/local/json.py        | 2 +-
 mopidy/local/translator.py  | 2 +-
 setup.cfg                   | 4 ++++
 tests/__init__.py           | 2 +-
 tests/utils/test_jsonrpc.py | 2 +-
 8 files changed, 12 insertions(+), 8 deletions(-)

diff --git a/mopidy/audio/actor.py b/mopidy/audio/actor.py
index ccb802a4..d72b364b 100644
--- a/mopidy/audio/actor.py
+++ b/mopidy/audio/actor.py
@@ -477,8 +477,8 @@ class Audio(pykka.ThreadingActor):
         playbin.set_property('flags', PLAYBIN_FLAGS)
 
         # TODO: turn into config values...
-        playbin.set_property('buffer-size', 2*1024*1024)
-        playbin.set_property('buffer-duration', 2*gst.SECOND)
+        playbin.set_property('buffer-size', 2 * 1024 * 1024)
+        playbin.set_property('buffer-duration', 2 * gst.SECOND)
 
         self._signals.connect(playbin, 'source-setup', self._on_source_setup)
         self._signals.connect(playbin, 'about-to-finish',
diff --git a/mopidy/audio/playlists.py b/mopidy/audio/playlists.py
index 5a362191..61bcb7a1 100644
--- a/mopidy/audio/playlists.py
+++ b/mopidy/audio/playlists.py
@@ -78,7 +78,7 @@ def parse_pls(data):
         if section.lower() != 'playlist':
             continue
         for i in range(cp.getint(section, 'numberofentries')):
-            yield cp.get(section, 'file%d' % (i+1))
+            yield cp.get(section, 'file%d' % (i + 1))
 
 
 def parse_xspf(data):
diff --git a/mopidy/local/__init__.py b/mopidy/local/__init__.py
index 73d07f75..62228e91 100644
--- a/mopidy/local/__init__.py
+++ b/mopidy/local/__init__.py
@@ -27,7 +27,7 @@ class Extension(ext.Extension):
         schema['playlists_dir'] = config.Path()
         schema['tag_cache_file'] = config.Deprecated()
         schema['scan_timeout'] = config.Integer(
-            minimum=1000, maximum=1000*60*60)
+            minimum=1000, maximum=1000 * 60 * 60)
         schema['scan_flush_threshold'] = config.Integer(minimum=0)
         schema['scan_follow_symlinks'] = config.Boolean()
         schema['excluded_file_extensions'] = config.List(optional=True)
diff --git a/mopidy/local/json.py b/mopidy/local/json.py
index 70dc68c4..38e1bf6c 100644
--- a/mopidy/local/json.py
+++ b/mopidy/local/json.py
@@ -75,7 +75,7 @@ class _BrowseCache(object):
             parent_uri = None
             child = None
             for i in reversed(range(len(parts))):
-                directory = '/'.join(parts[:i+1])
+                directory = '/'.join(parts[:i + 1])
                 uri = translator.path_to_local_directory_uri(directory)
 
                 # First dir we process is our parent
diff --git a/mopidy/local/translator.py b/mopidy/local/translator.py
index 3cbe2066..ab9fc28f 100644
--- a/mopidy/local/translator.py
+++ b/mopidy/local/translator.py
@@ -59,7 +59,7 @@ def m3u_extinf_to_track(line):
         return Track()
     (runtime, title) = m.groups()
     if int(runtime) > 0:
-        return Track(name=title, length=1000*int(runtime))
+        return Track(name=title, length=1000 * int(runtime))
     else:
         return Track(name=title)
 
diff --git a/setup.cfg b/setup.cfg
index 80ab9645..0d6c1486 100644
--- a/setup.cfg
+++ b/setup.cfg
@@ -1,6 +1,10 @@
 [flake8]
 application-import-names = mopidy,tests
 exclude = .git,.tox,build,js
+# Ignored flake8 warnings:
+# - E402 module level import not at top of file
+# - E731 do not assign a lambda expression, use a def
+ignore = E402,E731
 
 [wheel]
 universal = 1
diff --git a/tests/__init__.py b/tests/__init__.py
index 82759578..4283e604 100644
--- a/tests/__init__.py
+++ b/tests/__init__.py
@@ -22,7 +22,7 @@ class IsA(object):
         try:
             return isinstance(rhs, self.klass)
         except TypeError:
-            return type(rhs) == type(self.klass)
+            return type(rhs) == type(self.klass)  # flake8: noqa
 
     def __ne__(self, rhs):
         return not self.__eq__(rhs)
diff --git a/tests/utils/test_jsonrpc.py b/tests/utils/test_jsonrpc.py
index a74000b2..d236469e 100644
--- a/tests/utils/test_jsonrpc.py
+++ b/tests/utils/test_jsonrpc.py
@@ -614,7 +614,7 @@ class JsonRpcInspectorTest(JsonRpcTestBase):
             'core.library': core.LibraryController,
             'core.playback': core.PlaybackController,
             'core.playlists': core.PlaylistsController,
-            'core.tracklist':  core.TracklistController,
+            'core.tracklist': core.TracklistController,
         })
 
         methods = inspector.describe()