Fixed PEP8 issues except E501

devscripts/bash-completion.py

@@ -9,16 +9,17 @@ import youtube_dl
 BASH_COMPLETION_FILE = "youtube-dl.bash-completion"
 BASH_COMPLETION_TEMPLATE = "devscripts/bash-completion.in"
 
+
 def build_completion(opt_parser):
     opts_flag = []
     for group in opt_parser.option_groups:
         for option in group.option_list:
-            #for every long flag
+            # for every long flag
             opts_flag.append(option.get_opt_string())
     with open(BASH_COMPLETION_TEMPLATE) as f:
         template = f.read()
     with open(BASH_COMPLETION_FILE, "w") as f:
-        #just using the special char
+        # just using the special char
         filled_template = template.replace("{{flags}}", " ".join(opts_flag))
         f.write(filled_template)
 

devscripts/buildserver.py

@@ -233,7 +233,9 @@ def rmtree(path):
 
 #==============================================================================
 
+
 class BuildError(Exception):
+
     def __init__(self, output, code=500):
         self.output = output
         self.code = code
@@ -247,6 +249,7 @@ class HTTPError(BuildError):
 
 
 class PythonBuilder(object):
+
     def __init__(self, **kwargs):
         pythonVersion = kwargs.pop('python', '2.7')
         try:
@@ -262,6 +265,7 @@ class PythonBuilder(object):
 
 
 class GITInfoBuilder(object):
+
     def __init__(self, **kwargs):
         try:
             self.user, self.repoName = kwargs['path'][:2]
@@ -281,6 +285,7 @@ class GITInfoBuilder(object):
 
 
 class GITBuilder(GITInfoBuilder):
+
     def build(self):
         try:
             subprocess.check_output(['git', 'clone', 'git://github.com/%s/%s.git' % (self.user, self.repoName), self.buildPath])
@@ -313,6 +318,7 @@ class YoutubeDLBuilder(object):
 
 
 class DownloadBuilder(object):
+
     def __init__(self, **kwargs):
         self.handler = kwargs.pop('handler')
         self.srcPath = os.path.join(self.buildPath, *tuple(kwargs['path'][2:]))
@@ -341,6 +347,7 @@ class DownloadBuilder(object):
 
 
 class CleanupTempDir(object):
+
     def build(self):
         try:
             rmtree(self.basePath)
@@ -351,6 +358,7 @@ class CleanupTempDir(object):
 
 
 class Null(object):
+
     def __init__(self, **kwargs):
         pass
 
@@ -369,7 +377,7 @@ class Builder(PythonBuilder, GITBuilder, YoutubeDLBuilder, DownloadBuilder, Clea
 
 
 class BuildHTTPRequestHandler(BaseHTTPRequestHandler):
-    actionDict = { 'build': Builder, 'download': Builder } # They're the same, no more caching.
+    actionDict = {'build': Builder, 'download': Builder}  # They're the same, no more caching.
 
     def do_GET(self):
         path = urlparse.urlparse(self.path)

devscripts/gh-pages/update-feed.py

@@ -39,8 +39,7 @@ now_iso = now.isoformat() + 'Z'
 atom_template = atom_template.replace('@TIMESTAMP@', now_iso)
 
 versions_info = json.load(open('update/versions.json'))
-versions = list(versions_info['versions'].keys())
+versions = sorted(versions_info['versions'].keys())
-versions.sort()
 
 entries = []
 for v in versions:
@@ -73,4 +72,3 @@ atom_template = atom_template.replace('@ENTRIES@', entries_str)
 
 with io.open('update/releases.atom', 'w', encoding='utf-8') as atom_file:
     atom_file.write(atom_template)
-

devscripts/gh-pages/update-sites.py

@@ -9,6 +9,7 @@ sys.path.append(os.path.dirname(os.path.dirname(os.path.dirname(os.path.abspath(
 
 import youtube_dl
 
+
 def main():
     with open('supportedsites.html.in', 'r', encoding='utf-8') as tmplf:
         template = tmplf.read()
@@ -21,7 +22,7 @@ def main():
             continue
         elif ie_desc is not None:
             ie_html += ': {}'.format(ie.IE_DESC)
-        if ie.working() == False:
+        if not ie.working():
             ie_html += ' (Currently broken)'
         ie_htmls.append('<li>{}</li>'.format(ie_html))
 

devscripts/transition_helper.py

@@ -1,6 +1,7 @@
 #!/usr/bin/env python
 
-import sys, os
+import sys
+import os
 
 try:
     import urllib.request as compat_urllib_request

devscripts/transition_helper_exe/setup.py

@@ -9,4 +9,4 @@ py2exe_options = {
     "dll_excludes": ['w9xpopen.exe']
 }
 
-setup(console=['youtube-dl.py'], options={ "py2exe": py2exe_options }, zipfile=None)
+setup(console=['youtube-dl.py'], options={"py2exe": py2exe_options}, zipfile=None)

devscripts/transition_helper_exe/youtube-dl.py

@@ -1,17 +1,23 @@
 #!/usr/bin/env python
 
-import sys, os
+import sys
+import os
 import urllib2
-import json, hashlib
+import json
+import hashlib
+
 
 def rsa_verify(message, signature, key):
     from struct import pack
     from hashlib import sha256
     from sys import version_info
+
     def b(x):
-        if version_info[0] == 2: return x
+        if version_info[0] == 2:
-        else: return x.encode('latin1')
+            return x
-    assert(type(message) == type(b('')))
+        else:
+            return x.encode('latin1')
+    assert(isinstance(message, type(b(''))))
     block_size = 0
     n = key[0]
     while n:
@@ -23,13 +29,17 @@ def rsa_verify(message, signature, key):
         raw_bytes.insert(0, pack("B", signature & 0xFF))
         signature >>= 8
     signature = (block_size - len(raw_bytes)) * b('\x00') + b('').join(raw_bytes)
-    if signature[0:2] != b('\x00\x01'): return False
+    if signature[0:2] != b('\x00\x01'):
+        return False
     signature = signature[2:]
-    if not b('\x00') in signature: return False
+    if not b('\x00') in signature:
-    signature = signature[signature.index(b('\x00'))+1:]
+        return False
-    if not signature.startswith(b('\x30\x31\x30\x0D\x06\x09\x60\x86\x48\x01\x65\x03\x04\x02\x01\x05\x00\x04\x20')): return False
+    signature = signature[signature.index(b('\x00')) + 1:]
+    if not signature.startswith(b('\x30\x31\x30\x0D\x06\x09\x60\x86\x48\x01\x65\x03\x04\x02\x01\x05\x00\x04\x20')):
+        return False
     signature = signature[19:]
-    if signature != sha256(message).digest(): return False
+    if signature != sha256(message).digest():
+        return False
     return True
 
 sys.stderr.write(u'Hi! We changed distribution method and now youtube-dl needs to update itself one more time.\n')
@@ -92,7 +102,7 @@ echo Updating youtube-dl...
 ping 127.0.0.1 -n 5 -w 1000 > NUL
 move /Y "%s.new" "%s"
 del "%s"
-    \n""" %(exe, exe, bat))
+    \n""" % (exe, exe, bat))
     b.close()
 
     os.startfile(bat)

test/helper.py

@@ -47,6 +47,7 @@ def report_warning(message):
 
 
 class FakeYDL(YoutubeDL):
+
     def __init__(self, override=None):
         # Different instances of the downloader can't share the same dictionary
         # some test set the "sublang" parameter, which would break the md5 checks.
@@ -66,11 +67,14 @@ class FakeYDL(YoutubeDL):
     def expect_warning(self, regex):
         # Silence an expected warning matching a regex
         old_report_warning = self.report_warning
+
         def report_warning(self, message):
-            if re.match(regex, message): return
+            if re.match(regex, message):
+                return
             old_report_warning(message)
         self.report_warning = types.MethodType(report_warning, self)
 
+
 def get_testcases():
     for ie in youtube_dl.extractor.gen_extractors():
         t = getattr(ie, '_TEST', None)

test/test_YoutubeDL.py

@@ -14,6 +14,7 @@ from youtube_dl.extractor import YoutubeIE
 
 
 class YDL(FakeYDL):
+
     def __init__(self, *args, **kwargs):
         super(YDL, self).__init__(*args, **kwargs)
         self.downloaded_info_dicts = []
@@ -27,6 +28,7 @@ class YDL(FakeYDL):
 
 
 class TestFormatSelection(unittest.TestCase):
+
     def test_prefer_free_formats(self):
         # Same resolution => download webm
         ydl = YDL()
@@ -236,6 +238,7 @@ class TestFormatSelection(unittest.TestCase):
             'ext': 'mp4',
             'width': None,
         }
+
         def fname(templ):
             ydl = YoutubeDL({'outtmpl': templ})
             return ydl.prepare_filename(info)

test/test_age_restriction.py

@@ -32,6 +32,7 @@ def _download_restricted(url, filename, age):
 
 
 class TestAgeRestriction(unittest.TestCase):
+
     def _assert_restricted(self, url, filename, age, old_age=None):
         self.assertTrue(_download_restricted(url, filename, old_age))
         self.assertFalse(_download_restricted(url, filename, age))

test/test_all_urls.py

@@ -21,6 +21,7 @@ from youtube_dl.extractor import (
 
 
 class TestAllURLsMatching(unittest.TestCase):
+
     def setUp(self):
         self.ies = gen_extractors()
 
@@ -33,19 +34,19 @@ class TestAllURLsMatching(unittest.TestCase):
     def test_youtube_playlist_matching(self):
         assertPlaylist = lambda url: self.assertMatch(url, ['youtube:playlist'])
         assertPlaylist('ECUl4u3cNGP61MdtwGTqZA0MreSaDybji8')
-        assertPlaylist('UUBABnxM4Ar9ten8Mdjj1j0Q') #585
+        assertPlaylist('UUBABnxM4Ar9ten8Mdjj1j0Q')  # 585
         assertPlaylist('PL63F0C78739B09958')
         assertPlaylist('https://www.youtube.com/playlist?list=UUBABnxM4Ar9ten8Mdjj1j0Q')
         assertPlaylist('https://www.youtube.com/course?list=ECUl4u3cNGP61MdtwGTqZA0MreSaDybji8')
         assertPlaylist('https://www.youtube.com/playlist?list=PLwP_SiAcdui0KVebT0mU9Apz359a4ubsC')
-        assertPlaylist('https://www.youtube.com/watch?v=AV6J6_AeFEQ&playnext=1&list=PL4023E734DA416012') #668
+        assertPlaylist('https://www.youtube.com/watch?v=AV6J6_AeFEQ&playnext=1&list=PL4023E734DA416012')  # 668
         self.assertFalse('youtube:playlist' in self.matching_ies('PLtS2H6bU1M'))
         # Top tracks
         assertPlaylist('https://www.youtube.com/playlist?list=MCUS.20142101')
 
     def test_youtube_matching(self):
         self.assertTrue(YoutubeIE.suitable('PLtS2H6bU1M'))
-        self.assertFalse(YoutubeIE.suitable('https://www.youtube.com/watch?v=AV6J6_AeFEQ&playnext=1&list=PL4023E734DA416012')) #668
+        self.assertFalse(YoutubeIE.suitable('https://www.youtube.com/watch?v=AV6J6_AeFEQ&playnext=1&list=PL4023E734DA416012'))  # 668
         self.assertMatch('http://youtu.be/BaW_jenozKc', ['youtube'])
         self.assertMatch('http://www.youtube.com/v/BaW_jenozKc', ['youtube'])
         self.assertMatch('https://youtube.googleapis.com/v/BaW_jenozKc', ['youtube'])

test/test_download.py

@@ -34,18 +34,23 @@ from youtube_dl.extractor import get_info_extractor
 
 RETRIES = 3
 
+
 class YoutubeDL(youtube_dl.YoutubeDL):
+
     def __init__(self, *args, **kwargs):
         self.to_stderr = self.to_screen
         self.processed_info_dicts = []
         super(YoutubeDL, self).__init__(*args, **kwargs)
+
     def report_warning(self, message):
         # Don't accept warnings during tests
         raise ExtractorError(message)
+
     def process_info(self, info_dict):
         self.processed_info_dicts.append(info_dict)
         return super(YoutubeDL, self).process_info(info_dict)
 
+
 def _file_md5(fn):
     with open(fn, 'rb') as f:
         return hashlib.md5(f.read()).hexdigest()
@@ -55,15 +60,19 @@ defs = get_testcases()
 
 class TestDownload(unittest.TestCase):
     maxDiff = None
+
     def setUp(self):
         self.defs = defs
 
-### Dynamically generate tests
+# Dynamically generate tests
+
+
 def generator(test_case):
 
     def test_template(self):
         ie = youtube_dl.extractor.get_info_extractor(test_case['name'])
         other_ies = [get_info_extractor(ie_key) for ie_key in test_case.get('add_ie', [])]
+
         def print_skipping(reason):
             print('Skipping %s: %s' % (test_case['name'], reason))
         if not ie.working():
@@ -88,6 +97,7 @@ def generator(test_case):
         ydl = YoutubeDL(params)
         ydl.add_default_info_extractors()
         finished_hook_called = set()
+
         def _hook(status):
             if status['status'] == 'finished':
                 finished_hook_called.add(status['filename'])
@@ -97,6 +107,7 @@ def generator(test_case):
             return tc.get('file') or ydl.prepare_filename(tc.get('info_dict', {}))
 
         test_cases = test_case.get('playlist', [test_case])
+
         def try_rm_tcs_files():
             for tc in test_cases:
                 tc_filename = get_tc_filename(tc)
@@ -162,7 +173,7 @@ def generator(test_case):
 
     return test_template
 
-### And add them to TestDownload
+# And add them to TestDownload
 for n, test_case in enumerate(defs):
     test_method = generator(test_case)
     tname = 'test_' + str(test_case['name'])

test/test_execution.py

@@ -11,12 +11,14 @@ try:
 except AttributeError:
     _DEV_NULL = open(os.devnull, 'wb')
 
+
 class TestExecution(unittest.TestCase):
+
     def test_import(self):
         subprocess.check_call([sys.executable, '-c', 'import youtube_dl'], cwd=rootDir)
 
     def test_module_exec(self):
-        if sys.version_info >= (2,7): # Python 2.6 doesn't support package execution
+        if sys.version_info >= (2, 7):  # Python 2.6 doesn't support package execution
             subprocess.check_call([sys.executable, '-m', 'youtube_dl', '--version'], cwd=rootDir, stdout=_DEV_NULL)
 
     def test_main_exec(self):

test/test_playlists.py

@@ -40,6 +40,7 @@ from youtube_dl.extractor import (
 
 
 class TestPlaylists(unittest.TestCase):
+
     def assertIsPlaylist(self, info):
         """Make sure the info has '_type' set to 'playlist'"""
         self.assertEqual(info['_type'], 'playlist')

test/test_subtitles.py

@@ -21,6 +21,7 @@ from youtube_dl.extractor import (
 class BaseTestSubtitles(unittest.TestCase):
     url = None
     IE = None
+
     def setUp(self):
         self.DL = FakeYDL()
         self.ie = self.IE(self.DL)

test/test_unicode_literals.py

@@ -13,6 +13,7 @@ IGNORED_FILES = [
 
 
 class TestUnicodeLiterals(unittest.TestCase):
+
     def test_all_files(self):
         print('Skipping this test (not yet fully implemented)')
         return

test/test_utils.py

@@ -41,6 +41,7 @@ else:
 
 
 class TestUtil(unittest.TestCase):
+
     def test_timeconvert(self):
         self.assertTrue(timeconvert('') is None)
         self.assertTrue(timeconvert('bougrg') is None)
@@ -109,14 +110,14 @@ class TestUtil(unittest.TestCase):
         self.assertEqual(orderedSet([1, 1, 2, 3, 4, 4, 5, 6, 7, 3, 5]), [1, 2, 3, 4, 5, 6, 7])
         self.assertEqual(orderedSet([]), [])
         self.assertEqual(orderedSet([1]), [1])
-        #keep the list ordered
+        # keep the list ordered
         self.assertEqual(orderedSet([135, 1, 1, 1]), [135, 1])
 
     def test_unescape_html(self):
         self.assertEqual(unescapeHTML(_compat_str('%20;')), _compat_str('%20;'))
 
     def test_daterange(self):
-        _20century = DateRange("19000101","20000101")
+        _20century = DateRange("19000101", "20000101")
         self.assertFalse("17890714" in _20century)
         _ac = DateRange("00010101")
         self.assertTrue("19690721" in _ac)

test/test_write_annotations.py

@@ -19,6 +19,7 @@ import youtube_dl.extractor
 
 
 class YoutubeDL(youtube_dl.YoutubeDL):
+
     def __init__(self, *args, **kwargs):
         super(YoutubeDL, self).__init__(*args, **kwargs)
         self.to_stderr = self.to_screen
@@ -31,19 +32,19 @@ params = get_params({
 })
 
 
-
 TEST_ID = 'gr51aVj-mLg'
 ANNOTATIONS_FILE = TEST_ID + '.flv.annotations.xml'
 EXPECTED_ANNOTATIONS = ['Speech bubble', 'Note', 'Title', 'Spotlight', 'Label']
 
+
 class TestAnnotations(unittest.TestCase):
+
     def setUp(self):
         # Clear old files
         self.tearDown()
 
-
     def test_info_json(self):
-        expected = list(EXPECTED_ANNOTATIONS) #Two annotations could have the same text.
+        expected = list(EXPECTED_ANNOTATIONS)  # Two annotations could have the same text.
         ie = youtube_dl.extractor.YoutubeIE()
         ydl = YoutubeDL(params)
         ydl.add_info_extractor(ie)
@@ -59,19 +60,18 @@ class TestAnnotations(unittest.TestCase):
         self.assertEqual(annotationsTag.tag, 'annotations')
         annotations = annotationsTag.findall('annotation')
 
-        #Not all the annotations have TEXT children and the annotations are returned unsorted.
+        # Not all the annotations have TEXT children and the annotations are returned unsorted.
         for a in annotations:
                 self.assertEqual(a.tag, 'annotation')
                 if a.get('type') == 'text':
                         textTag = a.find('TEXT')
                         text = textTag.text
-                        self.assertTrue(text in expected) #assertIn only added in python 2.7
+                        self.assertTrue(text in expected)  # assertIn only added in python 2.7
-                        #remove the first occurance, there could be more than one annotation with the same text
+                        # remove the first occurance, there could be more than one annotation with the same text
                         expected.remove(text)
-        #We should have seen (and removed) all the expected annotation texts.
+        # We should have seen (and removed) all the expected annotation texts.
         self.assertEqual(len(expected), 0, 'Not all expected annotations were found.')
 
-
     def tearDown(self):
         try_rm(ANNOTATIONS_FILE)
 

test/test_write_info_json.py

@@ -18,6 +18,7 @@ import youtube_dl.extractor
 
 
 class YoutubeDL(youtube_dl.YoutubeDL):
+
     def __init__(self, *args, **kwargs):
         super(YoutubeDL, self).__init__(*args, **kwargs)
         self.to_stderr = self.to_screen
@@ -41,6 +42,7 @@ For more information, contact phihag@phihag.de .'''
 
 
 class TestInfoJSON(unittest.TestCase):
+
     def setUp(self):
         # Clear old files
         self.tearDown()

test/test_youtube_lists.py

@@ -20,6 +20,7 @@ from youtube_dl.extractor import (
 
 
 class TestYoutubeLists(unittest.TestCase):
+
     def assertIsPlaylist(self, info):
         """Make sure the info has '_type' set to 'playlist'"""
         self.assertEqual(info['_type'], 'playlist')
@@ -31,7 +32,7 @@ class TestYoutubeLists(unittest.TestCase):
         self.assertIsPlaylist(result)
         self.assertEqual(result['title'], 'ytdl test PL')
         ytie_results = [YoutubeIE().extract_id(url['url']) for url in result['entries']]
-        self.assertEqual(ytie_results, [ 'bV9L5Ht9LgY', 'FXxLjLQi3Fg', 'tU3Bgo5qJZE'])
+        self.assertEqual(ytie_results, ['bV9L5Ht9LgY', 'FXxLjLQi3Fg', 'tU3Bgo5qJZE'])
 
     def test_youtube_playlist_noplaylist(self):
         dl = FakeYDL()
@@ -55,7 +56,7 @@ class TestYoutubeLists(unittest.TestCase):
         self.assertTrue(len(result['entries']) >= 799)
 
     def test_youtube_playlist_with_deleted(self):
-        #651
+        # 651
         dl = FakeYDL()
         ie = YoutubePlaylistIE(dl)
         result = ie.extract('https://www.youtube.com/playlist?list=PLwP_SiAcdui0KVebT0mU9Apz359a4ubsC')
@@ -83,10 +84,10 @@ class TestYoutubeLists(unittest.TestCase):
     def test_youtube_channel(self):
         dl = FakeYDL()
         ie = YoutubeChannelIE(dl)
-        #test paginated channel
+        # test paginated channel
         result = ie.extract('https://www.youtube.com/channel/UCKfVa3S1e4PHvxWcwyMMg8w')
         self.assertTrue(len(result['entries']) > 90)
-        #test autogenerated channel
+        # test autogenerated channel
         result = ie.extract('https://www.youtube.com/channel/HCtnHdj3df7iM/videos')
         self.assertTrue(len(result['entries']) >= 18)
 

test/test_youtube_signature.py

@@ -37,6 +37,7 @@ _TESTS = [
 
 
 class TestSignature(unittest.TestCase):
+
     def setUp(self):
         TEST_DIR = os.path.dirname(os.path.abspath(__file__))
         self.TESTDATA_DIR = os.path.join(TEST_DIR, 'testdata')

youtube-dl

@@ -1,21 +1,27 @@
 #!/usr/bin/env python
 
-import sys, os
+import sys
-import json, hashlib
+import os
+import json
+import hashlib
 
 try:
     import urllib.request as compat_urllib_request
 except ImportError:  # Python 2
     import urllib2 as compat_urllib_request
 
+
 def rsa_verify(message, signature, key):
     from struct import pack
     from hashlib import sha256
     from sys import version_info
+
     def b(x):
-        if version_info[0] == 2: return x
+        if version_info[0] == 2:
-        else: return x.encode('latin1')
+            return x
-    assert(type(message) == type(b('')))
+        else:
+            return x.encode('latin1')
+    assert(isinstance(message, type(b(''))))
     block_size = 0
     n = key[0]
     while n:
@@ -27,13 +33,17 @@ def rsa_verify(message, signature, key):
         raw_bytes.insert(0, pack("B", signature & 0xFF))
         signature >>= 8
     signature = (block_size - len(raw_bytes)) * b('\x00') + b('').join(raw_bytes)
-    if signature[0:2] != b('\x00\x01'): return False
+    if signature[0:2] != b('\x00\x01'):
+        return False
     signature = signature[2:]
-    if not b('\x00') in signature: return False
+    if not b('\x00') in signature:
-    signature = signature[signature.index(b('\x00'))+1:]
+        return False
-    if not signature.startswith(b('\x30\x31\x30\x0D\x06\x09\x60\x86\x48\x01\x65\x03\x04\x02\x01\x05\x00\x04\x20')): return False
+    signature = signature[signature.index(b('\x00')) + 1:]
+    if not signature.startswith(b('\x30\x31\x30\x0D\x06\x09\x60\x86\x48\x01\x65\x03\x04\x02\x01\x05\x00\x04\x20')):
+        return False
     signature = signature[19:]
-    if signature != sha256(message).digest(): return False
+    if signature != sha256(message).digest():
+        return False
     return True
 
 sys.stderr.write(u'Hi! We changed distribution method and now youtube-dl needs to update itself one more time.\n')

youtube_dl/FileDownloader.py

@@ -5,6 +5,7 @@ from .downloader import get_suitable_downloader
 
 # This class reproduces the old behaviour of FileDownloader
 class FileDownloader(RealFileDownloader):
+
     def _do_download(self, filename, info_dict):
         real_fd = get_suitable_downloader(info_dict)(self.ydl, self.params)
         for ph in self._progress_hooks:

youtube_dl/YoutubeDL.py

@@ -61,6 +61,7 @@ from .version import __version__
 
 
 class YoutubeDL(object):
+
     """YoutubeDL class.
 
     YoutubeDL objects are the ones responsible of downloading the
@@ -268,7 +269,7 @@ class YoutubeDL(object):
             return message
 
         assert hasattr(self, '_output_process')
-        assert type(message) == type('')
+        assert isinstance(message, type(''))
         line_count = message.count('\n') + 1
         self._output_process.stdin.write((message + '\n').encode('utf-8'))
         self._output_process.stdin.flush()
@@ -293,7 +294,7 @@ class YoutubeDL(object):
 
     def to_stderr(self, message):
         """Print message to stderr."""
-        assert type(message) == type('')
+        assert isinstance(message, type(''))
         if self.params.get('logger'):
             self.params['logger'].error(message)
         else:
@@ -978,7 +979,7 @@ class YoutubeDL(object):
 
         for url in url_list:
             try:
-                #It also downloads the videos
+                # It also downloads the videos
                 self.extract_info(url)
             except UnavailableVideoError:
                 self.report_error('unable to download video')

youtube_dl/__init__.py

@@ -154,7 +154,8 @@ def parseOpts(overrideArguments=None):
         if len(opts) > 1:
             opts.insert(1, ', ')
 
-        if option.takes_value(): opts.append(' %s' % option.metavar)
+        if option.takes_value():
+            opts.append(' %s' % option.metavar)
 
         return "".join(opts)
 
@@ -166,7 +167,7 @@ def parseOpts(overrideArguments=None):
         for private_opt in ['-p', '--password', '-u', '--username', '--video-password']:
             try:
                 i = opts.index(private_opt)
-                opts[i+1] = '<PRIVATE>'
+                opts[i + 1] = '<PRIVATE>'
             except ValueError:
                 pass
         return opts
@@ -176,16 +177,17 @@ def parseOpts(overrideArguments=None):
 
     # No need to wrap help messages if we're on a wide console
     columns = get_term_width()
-    if columns: max_width = columns
+    if columns:
+        max_width = columns
 
     fmt = optparse.IndentedHelpFormatter(width=max_width, max_help_position=max_help_position)
     fmt.format_option_strings = _format_option_string
 
     kw = {
-        'version'   : __version__,
+        'version': __version__,
-        'formatter' : fmt,
+        'formatter': fmt,
-        'usage' : '%prog [options] url [url...]',
+        'usage': '%prog [options] url [url...]',
-        'conflict_handler' : 'resolve',
+        'conflict_handler': 'resolve',
     }
 
     parser = optparse.OptionParser(**kw)
@@ -250,7 +252,6 @@ def parseOpts(overrideArguments=None):
         action='store_true',
         help='Do not read configuration files. When given in the global configuration file /etc/youtube-dl.conf: do not read the user configuration in ~/.config/youtube-dl.conf (%APPDATA%/youtube-dl/config.txt on Windows)')
 
-
     selection.add_option(
         '--playlist-start',
         dest='playliststart', metavar='NUMBER', default=1, type=int,
@@ -259,8 +260,8 @@ def parseOpts(overrideArguments=None):
         '--playlist-end',
         dest='playlistend', metavar='NUMBER', default=None, type=int,
         help='playlist video to end at (default is last)')
-    selection.add_option('--match-title', dest='matchtitle', metavar='REGEX',help='download only matching titles (regex or caseless sub-string)')
+    selection.add_option('--match-title', dest='matchtitle', metavar='REGEX', help='download only matching titles (regex or caseless sub-string)')
-    selection.add_option('--reject-title', dest='rejecttitle', metavar='REGEX',help='skip download for matching titles (regex or caseless sub-string)')
+    selection.add_option('--reject-title', dest='rejecttitle', metavar='REGEX', help='skip download for matching titles (regex or caseless sub-string)')
     selection.add_option('--max-downloads', metavar='NUMBER',
                          dest='max_downloads', type=int, default=None,
                          help='Abort after downloading NUMBER files')
@@ -306,7 +307,6 @@ def parseOpts(overrideArguments=None):
     authentication.add_option('--video-password',
                               dest='videopassword', metavar='PASSWORD', help='video password (vimeo, smotri)')
 
-
     video_format.add_option('-f', '--format',
                             action='store', dest='format', metavar='FORMAT', default=None,
                             help='video format code, specify the order of preference using slashes: "-f 22/17/18". "-f mp4" and "-f flv" are also supported. You can also use the special names "best", "bestaudio", "worst", and "worstaudio". By default, youtube-dl will pick the best quality.')
@@ -402,7 +402,6 @@ def parseOpts(overrideArguments=None):
                          dest='debug_printtraffic', action='store_true', default=False,
                          help='Display sent and read HTTP traffic')
 
-
     filesystem.add_option('-t', '--title',
                           action='store_true', dest='usetitle', help='use title in file name (default)', default=False)
     filesystem.add_option('--id',
@@ -464,7 +463,6 @@ def parseOpts(overrideArguments=None):
                           action='store_true', dest='writethumbnail',
                           help='write thumbnail image to disk', default=False)
 
-
     postproc.add_option('-x', '--extract-audio', action='store_true', dest='extractaudio', default=False,
                         help='convert video files to audio-only files (requires ffmpeg or avconv and ffprobe or avprobe)')
     postproc.add_option('--audio-format', metavar='FORMAT', dest='audioformat', default='best',
@@ -488,7 +486,6 @@ def parseOpts(overrideArguments=None):
     postproc.add_option('--prefer-ffmpeg', action='store_true', dest='prefer_ffmpeg',
                         help='Prefer ffmpeg over avconv for running the postprocessors')
 
-
     parser.add_option_group(general)
     parser.add_option_group(selection)
     parser.add_option_group(downloader)
@@ -593,7 +590,6 @@ def _real_main(argv=None):
             compat_print(desc)
         sys.exit(0)
 
-
     # Conflicting, missing and erroneous options
     if opts.usenetrc and (opts.username is not None or opts.password is not None):
         parser.error(u'using .netrc conflicts with giving username/password')
@@ -657,14 +653,14 @@ def _real_main(argv=None):
 
     # --all-sub automatically sets --write-sub if --write-auto-sub is not given
     # this was the old behaviour if only --all-sub was given.
-    if opts.allsubtitles and (opts.writeautomaticsub == False):
+    if opts.allsubtitles and not opts.writeautomaticsub:
         opts.writesubtitles = True
 
     if sys.version_info < (3,):
         # In Python 2, sys.argv is a bytestring (also note http://bugs.python.org/issue2128 for Windows systems)
         if opts.outtmpl is not None:
             opts.outtmpl = opts.outtmpl.decode(preferredencoding())
-    outtmpl =((opts.outtmpl is not None and opts.outtmpl)
+    outtmpl = ((opts.outtmpl is not None and opts.outtmpl)
                or (opts.format == '-1' and opts.usetitle and u'%(title)s-%(id)s-%(format)s.%(ext)s')
                or (opts.format == '-1' and u'%(id)s-%(format)s.%(ext)s')
                or (opts.usetitle and opts.autonumber and u'%(autonumber)s-%(title)s-%(id)s.%(ext)s')

youtube_dl/aes.py

@@ -7,6 +7,7 @@ from .utils import bytes_to_intlist, intlist_to_bytes
 
 BLOCK_SIZE_BYTES = 16
 
+
 def aes_ctr_decrypt(data, key, counter):
     """
     Decrypt with aes in counter mode
@@ -20,11 +21,11 @@ def aes_ctr_decrypt(data, key, counter):
     expanded_key = key_expansion(key)
     block_count = int(ceil(float(len(data)) / BLOCK_SIZE_BYTES))
 
-    decrypted_data=[]
+    decrypted_data = []
     for i in range(block_count):
         counter_block = counter.next_value()
-        block = data[i*BLOCK_SIZE_BYTES : (i+1)*BLOCK_SIZE_BYTES]
+        block = data[i * BLOCK_SIZE_BYTES: (i + 1) * BLOCK_SIZE_BYTES]
-        block += [0]*(BLOCK_SIZE_BYTES - len(block))
+        block += [0] * (BLOCK_SIZE_BYTES - len(block))
 
         cipher_counter_block = aes_encrypt(counter_block, expanded_key)
         decrypted_data += xor(block, cipher_counter_block)
@@ -32,6 +33,7 @@ def aes_ctr_decrypt(data, key, counter):
 
     return decrypted_data
 
+
 def aes_cbc_decrypt(data, key, iv):
     """
     Decrypt with aes in CBC mode
@@ -44,11 +46,11 @@ def aes_cbc_decrypt(data, key, iv):
     expanded_key = key_expansion(key)
     block_count = int(ceil(float(len(data)) / BLOCK_SIZE_BYTES))
 
-    decrypted_data=[]
+    decrypted_data = []
     previous_cipher_block = iv
     for i in range(block_count):
-        block = data[i*BLOCK_SIZE_BYTES : (i+1)*BLOCK_SIZE_BYTES]
+        block = data[i * BLOCK_SIZE_BYTES: (i + 1) * BLOCK_SIZE_BYTES]
-        block += [0]*(BLOCK_SIZE_BYTES - len(block))
+        block += [0] * (BLOCK_SIZE_BYTES - len(block))
 
         decrypted_block = aes_decrypt(block, expanded_key)
         decrypted_data += xor(decrypted_block, previous_cipher_block)
@@ -57,6 +59,7 @@ def aes_cbc_decrypt(data, key, iv):
 
     return decrypted_data
 
+
 def key_expansion(data):
     """
     Generate key schedule
@@ -73,24 +76,25 @@ def key_expansion(data):
         temp = data[-4:]
         temp = key_schedule_core(temp, rcon_iteration)
         rcon_iteration += 1
-        data += xor(temp, data[-key_size_bytes : 4-key_size_bytes])
+        data += xor(temp, data[-key_size_bytes: 4 - key_size_bytes])
 
         for _ in range(3):
             temp = data[-4:]
-            data += xor(temp, data[-key_size_bytes : 4-key_size_bytes])
+            data += xor(temp, data[-key_size_bytes: 4 - key_size_bytes])
 
         if key_size_bytes == 32:
             temp = data[-4:]
             temp = sub_bytes(temp)
-            data += xor(temp, data[-key_size_bytes : 4-key_size_bytes])
+            data += xor(temp, data[-key_size_bytes: 4 - key_size_bytes])
 
         for _ in range(3 if key_size_bytes == 32 else 2 if key_size_bytes == 24 else 0):
             temp = data[-4:]
-            data += xor(temp, data[-key_size_bytes : 4-key_size_bytes])
+            data += xor(temp, data[-key_size_bytes: 4 - key_size_bytes])
     data = data[:expanded_key_size_bytes]
 
     return data
 
+
 def aes_encrypt(data, expanded_key):
     """
     Encrypt one block with aes
@@ -102,15 +106,16 @@ def aes_encrypt(data, expanded_key):
     rounds = len(expanded_key) // BLOCK_SIZE_BYTES - 1
 
     data = xor(data, expanded_key[:BLOCK_SIZE_BYTES])
-    for i in range(1, rounds+1):
+    for i in range(1, rounds + 1):
         data = sub_bytes(data)
         data = shift_rows(data)
         if i != rounds:
             data = mix_columns(data)
-        data = xor(data, expanded_key[i*BLOCK_SIZE_BYTES : (i+1)*BLOCK_SIZE_BYTES])
+        data = xor(data, expanded_key[i * BLOCK_SIZE_BYTES: (i + 1) * BLOCK_SIZE_BYTES])
 
     return data
 
+
 def aes_decrypt(data, expanded_key):
     """
     Decrypt one block with aes
@@ -122,7 +127,7 @@ def aes_decrypt(data, expanded_key):
     rounds = len(expanded_key) // BLOCK_SIZE_BYTES - 1
 
     for i in range(rounds, 0, -1):
-        data = xor(data, expanded_key[i*BLOCK_SIZE_BYTES : (i+1)*BLOCK_SIZE_BYTES])
+        data = xor(data, expanded_key[i * BLOCK_SIZE_BYTES: (i + 1) * BLOCK_SIZE_BYTES])
         if i != rounds:
             data = mix_columns_inv(data)
         data = shift_rows_inv(data)
@@ -131,6 +136,7 @@ def aes_decrypt(data, expanded_key):
 
     return data
 
+
 def aes_decrypt_text(data, password, key_size_bytes):
     """
     Decrypt text
@@ -149,14 +155,15 @@ def aes_decrypt_text(data, password, key_size_bytes):
     data = bytes_to_intlist(base64.b64decode(data))
     password = bytes_to_intlist(password.encode('utf-8'))
 
-    key = password[:key_size_bytes] + [0]*(key_size_bytes - len(password))
+    key = password[:key_size_bytes] + [0] * (key_size_bytes - len(password))
     key = aes_encrypt(key[:BLOCK_SIZE_BYTES], key_expansion(key)) * (key_size_bytes // BLOCK_SIZE_BYTES)
 
     nonce = data[:NONCE_LENGTH_BYTES]
     cipher = data[NONCE_LENGTH_BYTES:]
 
     class Counter:
-        __value = nonce + [0]*(BLOCK_SIZE_BYTES - NONCE_LENGTH_BYTES)
+        __value = nonce + [0] * (BLOCK_SIZE_BYTES - NONCE_LENGTH_BYTES)
+
         def next_value(self):
             temp = self.__value
             self.__value = inc(self.__value)
@@ -200,14 +207,14 @@ SBOX_INV = (0x52, 0x09, 0x6a, 0xd5, 0x30, 0x36, 0xa5, 0x38, 0xbf, 0x40, 0xa3, 0x
             0x60, 0x51, 0x7f, 0xa9, 0x19, 0xb5, 0x4a, 0x0d, 0x2d, 0xe5, 0x7a, 0x9f, 0x93, 0xc9, 0x9c, 0xef,
             0xa0, 0xe0, 0x3b, 0x4d, 0xae, 0x2a, 0xf5, 0xb0, 0xc8, 0xeb, 0xbb, 0x3c, 0x83, 0x53, 0x99, 0x61,
             0x17, 0x2b, 0x04, 0x7e, 0xba, 0x77, 0xd6, 0x26, 0xe1, 0x69, 0x14, 0x63, 0x55, 0x21, 0x0c, 0x7d)
-MIX_COLUMN_MATRIX = ((0x2,0x3,0x1,0x1),
+MIX_COLUMN_MATRIX = ((0x2, 0x3, 0x1, 0x1),
-                     (0x1,0x2,0x3,0x1),
+                     (0x1, 0x2, 0x3, 0x1),
-                     (0x1,0x1,0x2,0x3),
+                     (0x1, 0x1, 0x2, 0x3),
-                     (0x3,0x1,0x1,0x2))
+                     (0x3, 0x1, 0x1, 0x2))
-MIX_COLUMN_MATRIX_INV = ((0xE,0xB,0xD,0x9),
+MIX_COLUMN_MATRIX_INV = ((0xE, 0xB, 0xD, 0x9),
-                         (0x9,0xE,0xB,0xD),
+                         (0x9, 0xE, 0xB, 0xD),
-                         (0xD,0x9,0xE,0xB),
+                         (0xD, 0x9, 0xE, 0xB),
-                         (0xB,0xD,0x9,0xE))
+                         (0xB, 0xD, 0x9, 0xE))
 RIJNDAEL_EXP_TABLE = (0x01, 0x03, 0x05, 0x0F, 0x11, 0x33, 0x55, 0xFF, 0x1A, 0x2E, 0x72, 0x96, 0xA1, 0xF8, 0x13, 0x35,
                       0x5F, 0xE1, 0x38, 0x48, 0xD8, 0x73, 0x95, 0xA4, 0xF7, 0x02, 0x06, 0x0A, 0x1E, 0x22, 0x66, 0xAA,
                       0xE5, 0x34, 0x5C, 0xE4, 0x37, 0x59, 0xEB, 0x26, 0x6A, 0xBE, 0xD9, 0x70, 0x90, 0xAB, 0xE6, 0x31,
@@ -241,15 +248,19 @@ RIJNDAEL_LOG_TABLE = (0x00, 0x00, 0x19, 0x01, 0x32, 0x02, 0x1a, 0xc6, 0x4b, 0xc7
                       0x44, 0x11, 0x92, 0xd9, 0x23, 0x20, 0x2e, 0x89, 0xb4, 0x7c, 0xb8, 0x26, 0x77, 0x99, 0xe3, 0xa5,
                       0x67, 0x4a, 0xed, 0xde, 0xc5, 0x31, 0xfe, 0x18, 0x0d, 0x63, 0x8c, 0x80, 0xc0, 0xf7, 0x70, 0x07)
 
+
 def sub_bytes(data):
     return [SBOX[x] for x in data]
 
+
 def sub_bytes_inv(data):
     return [SBOX_INV[x] for x in data]
 
+
 def rotate(data):
     return data[1:] + [data[0]]
 
+
 def key_schedule_core(data, rcon_iteration):
     data = rotate(data)
     data = sub_bytes(data)
@@ -257,14 +268,17 @@ def key_schedule_core(data, rcon_iteration):
 
     return data
 
+
 def xor(data1, data2):
-    return [x^y for x, y in zip(data1, data2)]
+    return [x ^ y for x, y in zip(data1, data2)]
+
 
 def rijndael_mul(a, b):
-    if(a==0 or b==0):
+    if(a == 0 or b == 0):
         return 0
     return RIJNDAEL_EXP_TABLE[(RIJNDAEL_LOG_TABLE[a] + RIJNDAEL_LOG_TABLE[b]) % 0xFF]
 
+
 def mix_column(data, matrix):
     data_mixed = []
     for row in range(4):
@@ -275,33 +289,38 @@ def mix_column(data, matrix):
         data_mixed.append(mixed)
     return data_mixed
 
+
 def mix_columns(data, matrix=MIX_COLUMN_MATRIX):
     data_mixed = []
     for i in range(4):
-        column = data[i*4 : (i+1)*4]
+        column = data[i * 4: (i + 1) * 4]
         data_mixed += mix_column(column, matrix)
     return data_mixed
 
+
 def mix_columns_inv(data):
     return mix_columns(data, MIX_COLUMN_MATRIX_INV)
 
+
 def shift_rows(data):
     data_shifted = []
     for column in range(4):
         for row in range(4):
-            data_shifted.append( data[((column + row) & 0b11) * 4 + row] )
+            data_shifted.append(data[((column + row) & 0b11) * 4 + row])
     return data_shifted
 
+
 def shift_rows_inv(data):
     data_shifted = []
     for column in range(4):
         for row in range(4):
-            data_shifted.append( data[((column - row) & 0b11) * 4 + row] )
+            data_shifted.append(data[((column - row) & 0b11) * 4 + row])
     return data_shifted
 
+
 def inc(data):
     data = data[:]  # copy
-    for i in range(len(data)-1,-1,-1):
+    for i in range(len(data) - 1, -1, -1):
         if data[i] == 255:
             data[i] = 0
         else:

youtube_dl/downloader/common.py

@@ -11,6 +11,7 @@ from ..utils import (
 
 
 class FileDownloader(object):
+
     """File Downloader class.
 
     File downloader objects are the ones responsible of downloading the

youtube_dl/downloader/f4m.py

@@ -21,6 +21,7 @@ from ..utils import (
 
 
 class FlvReader(io.BytesIO):
+
     """
     Reader for Flv files
     The file format is documented in https://www.adobe.com/devnet/f4v.html
@@ -55,7 +56,7 @@ class FlvReader(io.BytesIO):
         if size == 1:
             real_size = self.read_unsigned_long_long()
             header_end = 16
-        return real_size, box_type, self.read(real_size-header_end)
+        return real_size, box_type, self.read(real_size - header_end)
 
     def read_asrt(self):
         # version
@@ -180,7 +181,7 @@ def build_fragments_list(boot_info):
     n_frags = segment_run_entry[1]
     fragment_run_entry_table = boot_info['fragments'][0]['fragments']
     first_frag_number = fragment_run_entry_table[0]['first']
-    for (i, frag_number) in zip(range(1, n_frags+1), itertools.count(first_frag_number)):
+    for (i, frag_number) in zip(range(1, n_frags + 1), itertools.count(first_frag_number)):
         res.append((1, frag_number))
     return res
 
@@ -210,11 +211,13 @@ def _add_ns(prop):
 
 
 class HttpQuietDownloader(HttpFD):
+
     def to_screen(self, *args, **kargs):
         pass
 
 
 class F4mFD(FileDownloader):
+
     """
     A downloader for f4m manifests or AdobeHDS.
     """

youtube_dl/downloader/hls.py

@@ -8,6 +8,7 @@ from ..utils import (
 
 
 class HlsFD(FileDownloader):
+
     def real_download(self, filename, info_dict):
         url = info_dict['url']
         self.report_destination(filename)

youtube_dl/downloader/http.py

@@ -14,6 +14,7 @@ from ..utils import (
 
 
 class HttpFD(FileDownloader):
+
     def real_download(self, filename, info_dict):
         url = info_dict['url']
         tmpfilename = self.temp_name(filename)

youtube_dl/downloader/mplayer.py

@@ -8,6 +8,7 @@ from ..utils import (
 
 
 class MplayerFD(FileDownloader):
+
     def real_download(self, filename, info_dict):
         url = info_dict['url']
         self.report_destination(filename)

youtube_dl/downloader/rtmp.py

@@ -12,6 +12,7 @@ from ..utils import (
 
 
 class RtmpFD(FileDownloader):
+
     def real_download(self, filename, info_dict):
         def run_rtmpdump(args):
             start = time.time()
@@ -36,13 +37,13 @@ class RtmpFD(FileDownloader):
                     continue
                 mobj = re.search(r'([0-9]+\.[0-9]{3}) kB / [0-9]+\.[0-9]{2} sec \(([0-9]{1,2}\.[0-9])%\)', line)
                 if mobj:
-                    downloaded_data_len = int(float(mobj.group(1))*1024)
+                    downloaded_data_len = int(float(mobj.group(1)) * 1024)
                     percent = float(mobj.group(2))
                     if not resume_percent:
                         resume_percent = percent
                         resume_downloaded_data_len = downloaded_data_len
-                    eta = self.calc_eta(start, time.time(), 100-resume_percent, percent-resume_percent)
+                    eta = self.calc_eta(start, time.time(), 100 - resume_percent, percent - resume_percent)
-                    speed = self.calc_speed(start, time.time(), downloaded_data_len-resume_downloaded_data_len)
+                    speed = self.calc_speed(start, time.time(), downloaded_data_len - resume_downloaded_data_len)
                     data_len = None
                     if percent > 0:
                         data_len = int(downloaded_data_len * 100 / percent)
@@ -62,7 +63,7 @@ class RtmpFD(FileDownloader):
                     # no percent for live streams
                     mobj = re.search(r'([0-9]+\.[0-9]{3}) kB / [0-9]+\.[0-9]{2} sec', line)
                     if mobj:
-                        downloaded_data_len = int(float(mobj.group(1))*1024)
+                        downloaded_data_len = int(float(mobj.group(1)) * 1024)
                         time_now = time.time()
                         speed = self.calc_speed(start, time_now, downloaded_data_len)
                         self.report_progress_live_stream(downloaded_data_len, speed, time_now - start)
@@ -78,7 +79,7 @@ class RtmpFD(FileDownloader):
                         if not cursor_in_new_line:
                             self.to_screen(u'')
                         cursor_in_new_line = True
-                        self.to_screen(u'[rtmpdump] '+line)
+                        self.to_screen(u'[rtmpdump] ' + line)
             proc.wait()
             if not cursor_in_new_line:
                 self.to_screen(u'')

youtube_dl/extractor/__init__.py

@@ -306,4 +306,4 @@ def gen_extractors():
 
 def get_info_extractor(ie_name):
     """Returns the info extractor class with the given ie_name"""
-    return globals()[ie_name+'IE']
+    return globals()[ie_name + 'IE']

youtube_dl/extractor/aparat.py

@@ -1,4 +1,4 @@
-#coding: utf-8
+# coding: utf-8
 
 import re
 
@@ -19,7 +19,7 @@ class AparatIE(InfoExtractor):
         u'info_dict': {
             u"title": u"تیم گلکسی 11 - زومیت",
         },
-        #u'skip': u'Extremely unreliable',
+        # u'skip': u'Extremely unreliable',
     }
 
     def _real_extract(self, url):

youtube_dl/extractor/appletrailers.py

@@ -66,11 +66,13 @@ class AppleTrailersIE(InfoExtractor):
         uploader_id = mobj.group('company')
 
         playlist_url = compat_urlparse.urljoin(url, u'includes/playlists/itunes.inc')
+
         def fix_html(s):
             s = re.sub(r'(?s)<script[^<]*?>.*?</script>', u'', s)
             s = re.sub(r'<img ([^<]*?)>', r'<img \1/>', s)
             # The ' in the onClick attributes are not escaped, it couldn't be parsed
             # like: http://trailers.apple.com/trailers/wb/gravity/
+
             def _clean_json(m):
                 return u'iTunes.playURL(%s);' % m.group(1).replace('\'', '&#39;')
             s = re.sub(self._JSON_RE, _clean_json, s)

youtube_dl/extractor/arte.py

@@ -19,6 +19,7 @@ from ..utils import (
 # is different for each one. The videos usually expire in 7 days, so we can't
 # add tests.
 
+
 class ArteTvIE(InfoExtractor):
     _VIDEOS_URL = r'(?:http://)?videos\.arte\.tv/(?P<lang>fr|de)/.*-(?P<id>.*?)\.html'
     _LIVEWEB_URL = r'(?:http://)?liveweb\.arte\.tv/(?P<lang>fr|de)/(?P<subpage>.+?)/(?P<name>.+)'
@@ -86,6 +87,7 @@ class ArteTvIE(InfoExtractor):
         config_xml = self._download_webpage(config_xml_url, video_id, note=u'Downloading configuration')
 
         video_urls = list(re.finditer(r'<url quality="(?P<quality>.*?)">(?P<url>.*?)</url>', config_xml))
+
         def _key(m):
             quality = m.group('quality')
             if quality == 'hd':
@@ -164,6 +166,7 @@ class ArteTVPlus7IE(InfoExtractor):
         all_formats = player_info['VSR'].values()
         # Some formats use the m3u8 protocol
         all_formats = list(filter(lambda f: f.get('videoFormat') != 'M3U8', all_formats))
+
         def _match_lang(f):
             if f.get('versionCode') is None:
                 return True
@@ -192,14 +195,15 @@ class ArteTVPlus7IE(InfoExtractor):
             def sort_key(f):
                 return (
                     # Sort first by quality
-                    int(f.get('height',-1)),
+                    int(f.get('height', -1)),
-                    int(f.get('bitrate',-1)),
+                    int(f.get('bitrate', -1)),
                     # The original version with subtitles has lower relevance
                     re.match(r'VO-ST(F|A)', f.get('versionCode', '')) is None,
                     # The version with sourds/mal subtitles has also lower relevance
                     re.match(r'VO?(F|A)-STM\1', f.get('versionCode', '')) is None,
                 )
         formats = sorted(formats, key=sort_key)
+
         def _format(format_info):
             quality = ''
             height = format_info.get('height')

youtube_dl/extractor/bambuser.py

@@ -18,7 +18,7 @@ class BambuserIE(InfoExtractor):
     _TEST = {
         'url': 'http://bambuser.com/v/4050584',
         # MD5 seems to be flaky, see https://travis-ci.org/rg3/youtube-dl/jobs/14051016#L388
-        #u'md5': 'fba8f7693e48fd4e8641b3fd5539a641',
+        # u'md5': 'fba8f7693e48fd4e8641b3fd5539a641',
         'info_dict': {
             'id': '4050584',
             'ext': 'flv',

youtube_dl/extractor/bandcamp.py

@@ -79,12 +79,12 @@ class BandcampIE(InfoExtractor):
         initial_url = mp3_info['url']
         re_url = r'(?P<server>http://(.*?)\.bandcamp\.com)/download/track\?enc=mp3-320&fsig=(?P<fsig>.*?)&id=(?P<id>.*?)&ts=(?P<ts>.*)$'
         m_url = re.match(re_url, initial_url)
-        #We build the url we will use to get the final track url
+        # We build the url we will use to get the final track url
         # This url is build in Bandcamp in the script download_bunde_*.js
         request_url = '%s/statdownload/track?enc=mp3-320&fsig=%s&id=%s&ts=%s&.rand=665028774616&.vrs=1' % (m_url.group('server'), m_url.group('fsig'), video_id, m_url.group('ts'))
         final_url_webpage = self._download_webpage(request_url, video_id, 'Requesting download url')
         # If we could correctly generate the .rand field the url would be
-        #in the "download_url" key
+        # in the "download_url" key
         final_url = re.search(r'"retry_url":"(.*?)"', final_url_webpage).group(1)
 
         return {

youtube_dl/extractor/bliptv.py

@@ -14,6 +14,7 @@ from ..utils import (
 
 
 class BlipTVIE(SubtitlesInfoExtractor):
+
     """Information extractor for blip.tv"""
 
     _VALID_URL = r'https?://(?:\w+\.)?blip\.tv/((.+/)|(play/)|(api\.swf#))(?P<presumptive_id>.+)$'

youtube_dl/extractor/canalplus.py

@@ -30,7 +30,7 @@ class CanalplusIE(InfoExtractor):
             webpage = self._download_webpage(url, mobj.group('path'))
             video_id = self._search_regex(r'videoId = "(\d+)";', webpage, u'video id')
         info_url = self._VIDEO_INFO_TEMPLATE % video_id
-        doc = self._download_xml(info_url,video_id, 
+        doc = self._download_xml(info_url, video_id,
                                  u'Downloading video info')
 
         self.report_extraction(video_id)

youtube_dl/extractor/channel9.py

@@ -5,7 +5,9 @@ import re
 from .common import InfoExtractor
 from ..utils import ExtractorError
 
+
 class Channel9IE(InfoExtractor):
+
     '''
     Common extractor for channel9.msdn.com.
 
@@ -31,7 +33,7 @@ class Channel9IE(InfoExtractor):
                 'session_code': 'KOS002',
                 'session_day': 'Day 1',
                 'session_room': 'Arena 1A',
-                'session_speakers': [ 'Ed Blankenship', 'Andrew Coates', 'Brady Gaster', 'Patrick Klug', 'Mads Kristensen' ],
+                'session_speakers': ['Ed Blankenship', 'Andrew Coates', 'Brady Gaster', 'Patrick Klug', 'Mads Kristensen'],
             },
         },
         {
@@ -44,7 +46,7 @@ class Channel9IE(InfoExtractor):
                 'description': 'md5:d1e6ecaafa7fb52a2cacdf9599829f5b',
                 'duration': 1540,
                 'thumbnail': 'http://media.ch9.ms/ch9/87e1/0300391f-a455-4c72-bec3-4422f19287e1/selfservicenuk_512.jpg',
-                'authors': [ 'Mike Wilmot' ],
+                'authors': ['Mike Wilmot'],
             },
         }
     ]
@@ -202,17 +204,17 @@ class Channel9IE(InfoExtractor):
 
         if slides is not None:
             d = common.copy()
-            d.update({ 'title': title + '-Slides', 'url': slides })
+            d.update({'title': title + '-Slides', 'url': slides})
             result.append(d)
 
         if zip_ is not None:
             d = common.copy()
-            d.update({ 'title': title + '-Zip', 'url': zip_ })
+            d.update({'title': title + '-Zip', 'url': zip_})
             result.append(d)
 
         if len(formats) > 0:
             d = common.copy()
-            d.update({ 'title': title, 'formats': formats })
+            d.update({'title': title, 'formats': formats})
             result.append(d)
 
         return result

youtube_dl/extractor/clipsyndicate.py

@@ -36,6 +36,7 @@ class ClipsyndicateIE(InfoExtractor):
             transform_source=fix_xml_ampersands)
 
         track_doc = pdoc.find('trackList/track')
+
         def find_param(name):
             node = find_xpath_attr(track_doc, './/param', 'name', name)
             if node is not None:

youtube_dl/extractor/cmt.py

@@ -1,5 +1,6 @@
 from .mtv import MTVIE
 
+
 class CMTIE(MTVIE):
     IE_NAME = u'cmt.com'
     _VALID_URL = r'https?://www\.cmt\.com/videos/.+?/(?P<videoid>[^/]+)\.jhtml'

youtube_dl/extractor/cnn.py

@@ -79,7 +79,7 @@ class CNNIE(InfoExtractor):
 
         self._sort_formats(formats)
 
-        thumbnails = sorted([((int(t.attrib['height']),int(t.attrib['width'])), t.text) for t in info.findall('images/image')])
+        thumbnails = sorted([((int(t.attrib['height']), int(t.attrib['width'])), t.text) for t in info.findall('images/image')])
         thumbs_dict = [{'resolution': res, 'url': t_url} for (res, t_url) in thumbnails]
 
         metas_el = info.find('metas')

youtube_dl/extractor/comedycentral.py

@@ -122,7 +122,7 @@ class ComedyCentralShowsIE(InfoExtractor):
                 epTitle = mobj.group('episode')
 
         self.report_extraction(epTitle)
-        webpage,htmlHandle = self._download_webpage_handle(url, epTitle)
+        webpage, htmlHandle = self._download_webpage_handle(url, epTitle)
         if dlNewest:
             url = htmlHandle.geturl()
             mobj = re.match(self._VALID_URL, url, re.VERBOSE)
@@ -154,7 +154,7 @@ class ComedyCentralShowsIE(InfoExtractor):
         results = []
 
         itemEls = idoc.findall('.//item')
-        for partNum,itemEl in enumerate(itemEls):
+        for partNum, itemEl in enumerate(itemEls):
             mediaId = itemEl.findall('./guid')[0].text
             shortMediaId = mediaId.split(':')[-1]
             showId = mediaId.split(':')[-2].replace('.com', '')
@@ -186,7 +186,7 @@ class ComedyCentralShowsIE(InfoExtractor):
                     'width': w,
                 })
 
-            effTitle = showId + '-' + epTitle + ' part ' + compat_str(partNum+1)
+            effTitle = showId + '-' + epTitle + ' part ' + compat_str(partNum + 1)
             results.append({
                 'id': shortMediaId,
                 'formats': formats,

youtube_dl/extractor/common.py

@@ -25,6 +25,7 @@ _NO_DEFAULT = object()
 
 
 class InfoExtractor(object):
+
     """Information Extractor class.
 
     Information extractors are the classes that, given a URL, extract
@@ -306,17 +307,18 @@ class InfoExtractor(object):
         """Report attempt to log in."""
         self.to_screen(u'Logging in')
 
-    #Methods for following #608
+    # Methods for following #608
     @staticmethod
     def url_result(url, ie=None, video_id=None):
         """Returns a url that points to a page that should be processed"""
-        #TODO: ie should be the class used for getting the info
+        # TODO: ie should be the class used for getting the info
         video_info = {'_type': 'url',
                       'url': url,
                       'ie_key': ie}
         if video_id is not None:
             video_info['id'] = video_id
         return video_info
+
     @staticmethod
     def playlist_result(entries, playlist_id=None, playlist_title=None):
         """Returns a playlist"""
@@ -340,7 +342,8 @@ class InfoExtractor(object):
         else:
             for p in pattern:
                 mobj = re.search(p, string, flags)
-                if mobj: break
+                if mobj:
+                    break
 
         if os.name != 'nt' and sys.stderr.isatty():
             _name = u'\033[0;34m%s\033[0m' % name
@@ -429,7 +432,8 @@ class InfoExtractor(object):
 
     def _og_search_video_url(self, html, name='video url', secure=True, **kargs):
         regexes = self._og_regexes('video')
-        if secure: regexes = self._og_regexes('video:secure_url') + regexes
+        if secure:
+            regexes = self._og_regexes('video:secure_url') + regexes
         return self._html_search_regex(regexes, html, name, **kargs)
 
     def _html_search_meta(self, name, html, display_name=None):
@@ -530,6 +534,7 @@ class InfoExtractor(object):
 
 
 class SearchInfoExtractor(InfoExtractor):
+
     """
     Base class for paged search queries extractors.
     They accept urls in the format _SEARCH_KEY(|all|[0-9]):{query}

youtube_dl/extractor/condenast.py

@@ -14,6 +14,7 @@ from ..utils import (
 
 
 class CondeNastIE(InfoExtractor):
+
     """
     Condé Nast is a media group, some of its sites use a custom HTML5 player
     that works the same in all of them.

youtube_dl/extractor/criterion.py

@@ -5,6 +5,7 @@ import re
 from .common import InfoExtractor
 from ..utils import determine_ext
 
+
 class CriterionIE(InfoExtractor):
     _VALID_URL = r'https?://www\.criterion\.com/films/(\d*)-.+'
     _TEST = {
@@ -13,7 +14,7 @@ class CriterionIE(InfoExtractor):
         u'md5': u'bc51beba55685509883a9a7830919ec3',
         u'info_dict': {
             u"title": u"Le Samouraï",
-            u"description" : u'md5:a2b4b116326558149bef81f76dcbb93f',
+            u"description": u'md5:a2b4b116326558149bef81f76dcbb93f',
         }
     }
 
@@ -32,7 +33,7 @@ class CriterionIE(InfoExtractor):
                                        webpage, 'thumbnail url')
 
         return {'id': video_id,
-                'url' : final_url,
+                'url': final_url,
                 'title': title,
                 'ext': determine_ext(final_url),
                 'description': description,

youtube_dl/extractor/crunchyroll.py

@@ -1,7 +1,9 @@
 # encoding: utf-8
 from __future__ import unicode_literals
 
-import re, base64, zlib
+import re
+import base64
+import zlib
 from hashlib import sha1
 from math import pow, sqrt, floor
 from .common import InfoExtractor
@@ -19,6 +21,7 @@ from ..aes import (
     inc,
 )
 
+
 class CrunchyrollIE(InfoExtractor):
     _VALID_URL = r'(?:https?://)?(?:(?P<prefix>www|m)\.)?(?P<url>crunchyroll\.com/(?:[^/]*/[^/?&]*?|media/\?id=)(?P<video_id>[0-9]+))(?:[/?&]|$)'
     _TESTS = [{
@@ -70,8 +73,10 @@ class CrunchyrollIE(InfoExtractor):
             return shaHash + [0] * 12
 
         key = obfuscate_key(id)
+
         class Counter:
             __value = iv
+
             def next_value(self):
                 temp = self.__value
                 self.__value = inc(self.__value)
@@ -80,7 +85,7 @@ class CrunchyrollIE(InfoExtractor):
         return zlib.decompress(decrypted_data)
 
     def _convert_subtitles_to_srt(self, subtitles):
-        i=1
+        i = 1
         output = ''
         for start, end, text in re.findall(r'<event [^>]*?start="([^"]+)" [^>]*?end="([^"]+)" [^>]*?text="([^"]+)"[^>]*?>', subtitles):
             start = start.replace('.', ',')
@@ -90,10 +95,10 @@ class CrunchyrollIE(InfoExtractor):
             if not text:
                 continue
             output += '%d\n%s --> %s\n%s\n\n' % (i, start, end, text)
-            i+=1
+            i += 1
         return output
 
-    def _real_extract(self,url):
+    def _real_extract(self, url):
         mobj = re.match(self._VALID_URL, url)
         video_id = mobj.group('video_id')
 
@@ -130,13 +135,13 @@ class CrunchyrollIE(InfoExtractor):
         formats = []
         for fmt in re.findall(r'\?p([0-9]{3,4})=1', webpage):
             stream_quality, stream_format = self._FORMAT_IDS[fmt]
-            video_format = fmt+'p'
+            video_format = fmt + 'p'
             streamdata_req = compat_urllib_request.Request('http://www.crunchyroll.com/xml/')
             # urlencode doesn't work!
-            streamdata_req.data = 'req=RpcApiVideoEncode%5FGetStreamInfo&video%5Fencode%5Fquality='+stream_quality+'&media%5Fid='+stream_id+'&video%5Fformat='+stream_format
+            streamdata_req.data = 'req=RpcApiVideoEncode%5FGetStreamInfo&video%5Fencode%5Fquality=' + stream_quality + '&media%5Fid=' + stream_id + '&video%5Fformat=' + stream_format
             streamdata_req.add_header('Content-Type', 'application/x-www-form-urlencoded')
             streamdata_req.add_header('Content-Length', str(len(streamdata_req.data)))
-            streamdata = self._download_webpage(streamdata_req, video_id, note='Downloading media info for '+video_format)
+            streamdata = self._download_webpage(streamdata_req, video_id, note='Downloading media info for ' + video_format)
             video_url = self._search_regex(r'<host>([^<]+)', streamdata, 'video_url')
             video_play_path = self._search_regex(r'<file>([^<]+)', streamdata, 'video_play_path')
             formats.append({
@@ -149,8 +154,8 @@ class CrunchyrollIE(InfoExtractor):
 
         subtitles = {}
         for sub_id, sub_name in re.findall(r'\?ssid=([0-9]+)" title="([^"]+)', webpage):
-            sub_page = self._download_webpage('http://www.crunchyroll.com/xml/?req=RpcApiSubtitle_GetXml&subtitle_script_id='+sub_id,\
+            sub_page = self._download_webpage('http://www.crunchyroll.com/xml/?req=RpcApiSubtitle_GetXml&subtitle_script_id=' + sub_id,
-                                              video_id, note='Downloading subtitles for '+sub_name)
+                                              video_id, note='Downloading subtitles for ' + sub_name)
             id = self._search_regex(r'id=\'([0-9]+)', sub_page, 'subtitle_id', fatal=False)
             iv = self._search_regex(r'<iv>([^<]+)', sub_page, 'subtitle_iv', fatal=False)
             data = self._search_regex(r'<data>([^<]+)', sub_page, 'subtitle_data', fatal=False)

youtube_dl/extractor/dailymotion.py

@@ -16,7 +16,9 @@ from ..utils import (
     ExtractorError,
 )
 
+
 class DailymotionBaseInfoExtractor(InfoExtractor):
+
     @staticmethod
     def _build_request(url):
         """Build a request with the family filter disabled"""
@@ -25,7 +27,9 @@ class DailymotionBaseInfoExtractor(InfoExtractor):
         request.add_header('Cookie', 'ff=off')
         return request
 
+
 class DailymotionIE(DailymotionBaseInfoExtractor, SubtitlesInfoExtractor):
+
     """Information Extractor for Dailymotion"""
 
     _VALID_URL = r'(?i)(?:https?://)?(?:(www|touch)\.)?dailymotion\.[a-z]{2,3}/(?:(embed|#)/)?video/(?P<id>[^/?_]+)'

youtube_dl/extractor/depositfiles.py

@@ -15,6 +15,7 @@ from ..utils import (
 
 
 class DepositFilesIE(InfoExtractor):
+
     """Information extractor for depositfiles.com"""
 
     _VALID_URL = r'(?:http://)?(?:\w+\.)?depositfiles\.com/(?:../(?#locale))?files/(.+)'
@@ -25,7 +26,7 @@ class DepositFilesIE(InfoExtractor):
         url = 'http://depositfiles.com/en/files/' + file_id
 
         # Retrieve file webpage with 'Free download' button pressed
-        free_download_indication = {'gateway_result' : '1'}
+        free_download_indication = {'gateway_result': '1'}
         request = compat_urllib_request.Request(url, compat_urllib_parse.urlencode(free_download_indication))
         try:
             self.report_download_webpage(file_id)

youtube_dl/extractor/dotsub.py

@@ -27,7 +27,7 @@ class DotsubIE(InfoExtractor):
         video_id = mobj.group('id')
         info_url = "https://dotsub.com/api/media/%s/metadata" % video_id
         info = self._download_json(info_url, video_id)
-        date = time.gmtime(info['dateCreated']/1000) # The timestamp is in miliseconds
+        date = time.gmtime(info['dateCreated'] / 1000)  # The timestamp is in miliseconds
 
         return {
             'id': video_id,

youtube_dl/extractor/dreisat.py

@@ -23,7 +23,6 @@ class DreiSatIE(InfoExtractor):
         }
     }
 
-
     def _real_extract(self, url):
         mobj = re.match(self._VALID_URL, url)
         video_id = mobj.group('id')

youtube_dl/extractor/ehow.py

@@ -43,4 +43,3 @@ class EHowIE(InfoExtractor):
             'description': self._og_search_description(webpage),
             'uploader': uploader,
         }
-

youtube_dl/extractor/eighttracks.py

@@ -82,7 +82,6 @@ class EightTracksIE(InfoExtractor):
         ]
     }
 
-
     def _real_extract(self, url):
         mobj = re.match(self._VALID_URL, url)
         if mobj is None:
@@ -102,7 +101,7 @@ class EightTracksIE(InfoExtractor):
         res = []
         for i in range(track_count):
             api_json = self._download_webpage(next_url, playlist_id,
-                note=u'Downloading song information %s/%s' % (str(i+1), track_count),
+                                              note=u'Downloading song information %s/%s' % (str(i + 1), track_count),
                                               errnote=u'Failed to download song information')
             api_data = json.loads(api_json)
             track_data = api_data[u'set']['track']

youtube_dl/extractor/extremetube.py

@@ -8,6 +8,7 @@ from ..utils import (
     compat_urllib_parse,
 )
 
+
 class ExtremeTubeIE(InfoExtractor):
     _VALID_URL = r'^(?:https?://)?(?:www\.)?(?P<url>extremetube\.com/video/.+?(?P<videoid>[0-9]+))(?:[/?&]|$)'
     _TEST = {

youtube_dl/extractor/facebook.py

@@ -15,6 +15,7 @@ from ..utils import (
 
 
 class FacebookIE(InfoExtractor):
+
     """Information Extractor for Facebook"""
 
     _VALID_URL = r'''(?x)

youtube_dl/extractor/flickr.py

@@ -10,6 +10,7 @@ from ..utils import (
 
 
 class FlickrIE(InfoExtractor):
+
     """Information Extractor for Flickr videos"""
     _VALID_URL = r'(?:https?://)?(?:www\.|secure\.)?flickr\.com/photos/(?P<uploader_id>[\w\-_@]+)/(?P<id>\d+).*'
     _TEST = {

youtube_dl/extractor/francetv.py

@@ -12,6 +12,7 @@ from ..utils import (
 
 
 class FranceTVBaseInfoExtractor(InfoExtractor):
+
     def _extract_video(self, video_id):
         info = self._download_xml(
             'http://www.francetvinfo.fr/appftv/webservices/video/'

youtube_dl/extractor/gamekings.py

@@ -9,7 +9,7 @@ class GamekingsIE(InfoExtractor):
         u"url": u"http://www.gamekings.tv/videos/phoenix-wright-ace-attorney-dual-destinies-review/",
         u'file': u'20130811.mp4',
         # MD5 is flaky, seems to change regularly
-        #u'md5': u'2f32b1f7b80fdc5cb616efb4f387f8a3',
+        # u'md5': u'2f32b1f7b80fdc5cb616efb4f387f8a3',
         u'info_dict': {
             u"title": u"Phoenix Wright: Ace Attorney \u2013 Dual Destinies Review",
             u"description": u"Melle en Steven hebben voor de review een week in de rechtbank doorbracht met Phoenix Wright: Ace Attorney - Dual Destinies.",

youtube_dl/extractor/generic.py

@@ -116,14 +116,16 @@ class GenericIE(InfoExtractor):
         """Check if it is a redirect, like url shorteners, in case return the new url."""
 
         class HEADRedirectHandler(compat_urllib_request.HTTPRedirectHandler):
+
             """
             Subclass the HTTPRedirectHandler to make it use our
             HEADRequest also on the redirected URL
             """
+
             def redirect_request(self, req, fp, code, msg, headers, newurl):
                 if code in (301, 302, 303, 307):
                     newurl = newurl.replace(' ', '%20')
-                    newheaders = dict((k,v) for k,v in req.headers.items()
+                    newheaders = dict((k, v) for k, v in req.headers.items()
                                       if k.lower() not in ("content-length", "content-type"))
                     return HEADRequest(newurl,
                                        headers=newheaders,
@@ -133,14 +135,16 @@ class GenericIE(InfoExtractor):
                     raise compat_urllib_error.HTTPError(req.get_full_url(), code, msg, headers, fp)
 
         class HTTPMethodFallback(compat_urllib_request.BaseHandler):
+
             """
             Fallback to GET if HEAD is not allowed (405 HTTP error)
             """
+
             def http_error_405(self, req, fp, code, msg, headers):
                 fp.read()
                 fp.close()
 
-                newheaders = dict((k,v) for k,v in req.headers.items()
+                newheaders = dict((k, v) for k, v in req.headers.items()
                                   if k.lower() not in ("content-length", "content-type"))
                 return self.parent.open(compat_urllib_request.Request(req.get_full_url(),
                                                                       headers=newheaders,
@@ -301,7 +305,7 @@ class GenericIE(InfoExtractor):
         # Look for embedded blip.tv player
         mobj = re.search(r'<meta\s[^>]*https?://api\.blip\.tv/\w+/redirect/\w+/(\d+)', webpage)
         if mobj:
-            return self.url_result('http://blip.tv/a/a-'+mobj.group(1), 'BlipTV')
+            return self.url_result('http://blip.tv/a/a-' + mobj.group(1), 'BlipTV')
         mobj = re.search(r'<(?:iframe|embed|object)\s[^>]*(https?://(?:\w+\.)?blip\.tv/(?:play/|api\.swf#)[a-zA-Z0-9]+)', webpage)
         if mobj:
             return self.url_result(mobj.group(1), 'BlipTV')

youtube_dl/extractor/hark.py

@@ -6,6 +6,7 @@ import json
 from .common import InfoExtractor
 from ..utils import determine_ext
 
+
 class HarkIE(InfoExtractor):
     _VALID_URL = r'https?://www\.hark\.com/clips/(.+?)-.+'
     _TEST = {
@@ -22,13 +23,13 @@ class HarkIE(InfoExtractor):
     def _real_extract(self, url):
         mobj = re.match(self._VALID_URL, url)
         video_id = mobj.group(1)
-        json_url = "http://www.hark.com/clips/%s.json" %(video_id)
+        json_url = "http://www.hark.com/clips/%s.json" % (video_id)
         info_json = self._download_webpage(json_url, video_id)
         info = json.loads(info_json)
         final_url = info['url']
 
         return {'id': video_id,
-                'url' : final_url,
+                'url': final_url,
                 'title': info['name'],
                 'ext': determine_ext(final_url),
                 'description': info['description'],

youtube_dl/extractor/hypem.py

@@ -13,6 +13,7 @@ from ..utils import (
 
 
 class HypemIE(InfoExtractor):
+
     """Information Extractor for hypem"""
     _VALID_URL = r'(?:http://)?(?:www\.)?hypem\.com/track/([^/]+)/([^/]+)'
     _TEST = {
@@ -40,7 +41,7 @@ class HypemIE(InfoExtractor):
         self.report_extraction(track_id)
 
         html_tracks = self._html_search_regex(r'<script type="application/json" id="displayList-data">(.*?)</script>',
-            response, u'tracks', flags=re.MULTILINE|re.DOTALL).strip()
+                                              response, u'tracks', flags=re.MULTILINE | re.DOTALL).strip()
         try:
             track_list = json.loads(html_tracks)
             track = track_list[u'tracks'][0]
@@ -53,7 +54,7 @@ class HypemIE(InfoExtractor):
         title = track[u"song"]
 
         serve_url = "http://hypem.com/serve/source/%s/%s" % (compat_str(track_id), compat_str(key))
-        request = compat_urllib_request.Request(serve_url, "" , {'Content-Type': 'application/json'})
+        request = compat_urllib_request.Request(serve_url, "", {'Content-Type': 'application/json'})
         request.add_header('cookie', cookie)
         song_data_json = self._download_webpage(request, track_id, u'Downloading metadata')
         try:

youtube_dl/extractor/ign.py

@@ -8,6 +8,7 @@ from ..utils import (
 
 
 class IGNIE(InfoExtractor):
+
     """
     Extractor for some of the IGN sites, like www.ign.com, es.ign.com de.ign.com.
     Some videos of it.ign.com are also supported
@@ -101,6 +102,7 @@ class IGNIE(InfoExtractor):
 
 
 class OneUPIE(IGNIE):
+
     """Extractor for 1up.com, it uses the ign videos system."""
 
     _VALID_URL = r'https?://gamevideos\.1up\.com/(?P<type>video)/id/(?P<name_or_id>.+)'

youtube_dl/extractor/internetvideoarchive.py

@@ -29,7 +29,7 @@ class InternetVideoArchiveIE(InfoExtractor):
     def _clean_query(query):
         NEEDED_ARGS = ['publishedid', 'customerid']
         query_dic = compat_urlparse.parse_qs(query)
-        cleaned_dic = dict((k,v[0]) for (k,v) in query_dic.items() if k in NEEDED_ARGS)
+        cleaned_dic = dict((k, v[0]) for (k, v) in query_dic.items() if k in NEEDED_ARGS)
         # Other player ids return m3u8 urls
         cleaned_dic['playerid'] = '247'
         cleaned_dic['videokbrate'] = '100000'

youtube_dl/extractor/iprima.py

@@ -34,8 +34,8 @@ class IPrimaIE(InfoExtractor):
         webpage = self._download_webpage(url, video_id)
 
         player_url = 'http://embed.livebox.cz/iprimaplay/player-embed-v2.js?__tok%s__=%s' % (
-                         floor(random()*1073741824),
+            floor(random() * 1073741824),
-                         floor(random()*1073741824))
+            floor(random() * 1073741824))
 
         req = compat_urllib_request.Request(player_url)
         req.add_header('Referer', url)
@@ -46,7 +46,7 @@ class IPrimaIE(InfoExtractor):
         zoneGEO = self._html_search_regex(r'"zoneGEO":(.+?),', webpage, 'zoneGEO')
 
         if zoneGEO != '0':
-            base_url = base_url.replace('token', 'token_'+zoneGEO)
+            base_url = base_url.replace('token', 'token_' + zoneGEO)
 
         formats = []
         for format_id in ['lq', 'hq', 'hd']:
@@ -63,13 +63,13 @@ class IPrimaIE(InfoExtractor):
                 quality = 1
             elif format_id == 'hd':
                 quality = 2
-                filename = 'hq/'+filename
+                filename = 'hq/' + filename
 
             formats.append({
                 'format_id': format_id,
                 'url': base_url,
                 'quality': quality,
-                'play_path': 'mp4:'+filename.replace('"', '')[:-4],
+                'play_path': 'mp4:' + filename.replace('"', '')[:-4],
                 'rtmp_live': True,
                 'ext': 'flv',
             })

youtube_dl/extractor/jadorecettepub.py

@@ -45,4 +45,3 @@ class JadoreCettePubIE(InfoExtractor):
             'title': title,
             'description': description,
         }
-

youtube_dl/extractor/jukebox.py

@@ -7,6 +7,7 @@ from ..utils import (
     unescapeHTML,
 )
 
+
 class JukeboxIE(InfoExtractor):
     _VALID_URL = r'^http://www\.jukebox?\..+?\/.+[,](?P<video_id>[a-z0-9\-]+)\.html'
     _IFRAME = r'<iframe .*src="(?P<iframe>[^"]*)".*>'
@@ -37,10 +38,10 @@ class JukeboxIE(InfoExtractor):
             mobj = re.search(self._IS_YOUTUBE, iframe_html)
             if mobj is None:
                 raise ExtractorError(u'Cannot extract video url')
-            youtube_url = unescapeHTML(mobj.group('youtube_url')).replace('\/','/')
+            youtube_url = unescapeHTML(mobj.group('youtube_url')).replace('\/', '/')
             self.to_screen(u'Youtube video detected')
-            return self.url_result(youtube_url,ie='Youtube')
+            return self.url_result(youtube_url, ie='Youtube')
-        video_url = unescapeHTML(mobj.group('video_url')).replace('\/','/')
+        video_url = unescapeHTML(mobj.group('video_url')).replace('\/', '/')
         video_ext = unescapeHTML(mobj.group('video_ext'))
 
         mobj = re.search(self._TITLE, html)

youtube_dl/extractor/justintv.py

@@ -10,6 +10,7 @@ from ..utils import (
 
 
 class JustinTVIE(InfoExtractor):
+
     """Information extractor for justin.tv and twitch.tv"""
     # TODO: One broadcast may be split into multiple videos. The key
     # 'broadcast_id' is the same for all parts, and 'broadcast_part'
@@ -49,7 +50,7 @@ class JustinTVIE(InfoExtractor):
                                            u'unable to download video info JSON')
 
         response = json.loads(info_json)
-        if type(response) != list:
+        if not isinstance(response, list):
             error_text = response.get('error', 'unknown error')
             raise ExtractorError(u'Justin.tv API: %s' % error_text)
         info = []
@@ -116,7 +117,7 @@ class JustinTVIE(InfoExtractor):
 
             # TODO determine start (and probably fix up file)
             #  youtube-dl -v http://www.twitch.tv/firmbelief/c/1757457
-            #video_url += u'?start=' + TODO:start_timestamp
+            # video_url += u'?start=' + TODO:start_timestamp
             # bracket_start is 13290, but we want 51670615
             self._downloader.report_warning(u'Chapter detected, but we can just download the whole file. '
                                             u'Chapter starts at %s and ends at %s' % (formatSeconds(bracket_start), formatSeconds(bracket_end)))

youtube_dl/extractor/keezmovies.py

@@ -11,6 +11,7 @@ from ..aes import (
     aes_decrypt_text
 )
 
+
 class KeezMoviesIE(InfoExtractor):
     _VALID_URL = r'^(?:https?://)?(?:www\.)?(?P<url>keezmovies\.com/video/.+?(?P<videoid>[0-9]+))(?:[/?&]|$)'
     _TEST = {
@@ -40,7 +41,7 @@ class KeezMoviesIE(InfoExtractor):
 
         video_title = self._html_search_regex(r'<h1 [^>]*>([^<]+)', webpage, u'title')
         video_url = compat_urllib_parse.unquote(self._html_search_regex(r'video_url=(.+?)&amp;', webpage, u'video_url'))
-        if webpage.find('encrypted=true')!=-1:
+        if webpage.find('encrypted=true') != -1:
             password = self._html_search_regex(r'video_title=(.+?)&amp;', webpage, u'password')
             video_url = aes_decrypt_text(video_url, password, 32).decode('utf-8')
         path = compat_urllib_parse_urlparse(video_url).path

youtube_dl/extractor/lifenews.py

@@ -53,7 +53,7 @@ class LifeNewsIE(InfoExtractor):
             r'<div class=\'comments\'>(\d+)</div>', webpage, 'comment count', fatal=False)
 
         upload_date = self._html_search_regex(
-            r'<time datetime=\'([^\']+)\'>', webpage, 'upload date',fatal=False)
+            r'<time datetime=\'([^\']+)\'>', webpage, 'upload date', fatal=False)
         if upload_date is not None:
             upload_date = unified_strdate(upload_date)
 

youtube_dl/extractor/livestream.py

@@ -29,7 +29,7 @@ class LivestreamIE(InfoExtractor):
                 'ext': 'mp4',
                 'title': video_data['caption'],
                 'thumbnail': video_data['thumbnail_url'],
-                'upload_date': video_data['updated_at'].replace('-','')[:8],
+                'upload_date': video_data['updated_at'].replace('-', '')[:8],
                 }
 
     def _real_extract(self, url):

youtube_dl/extractor/malemotion.py

@@ -7,6 +7,7 @@ from ..utils import (
     compat_urllib_parse,
 )
 
+
 class MalemotionIE(InfoExtractor):
     _VALID_URL = r'^(?:https?://)?malemotion\.com/video/(.+?)\.(?P<id>.+?)(#|$)'
     _TEST = {

youtube_dl/extractor/metacafe.py

@@ -9,7 +9,9 @@ from ..utils import (
     ExtractorError,
 )
 
+
 class MetacafeIE(InfoExtractor):
+
     """Information Extractor for metacafe.com."""
 
     _VALID_URL = r'(?:http://)?(?:www\.)?metacafe\.com/watch/([^/]+)/([^/]+)/.*'
@@ -82,7 +84,6 @@ class MetacafeIE(InfoExtractor):
         },
     ]
 
-
     def report_disclaimer(self):
         """Report disclaimer retrieval."""
         self.to_screen(u'Retrieving disclaimer')

youtube_dl/extractor/mofosex.py

@@ -8,6 +8,7 @@ from ..utils import (
     compat_urllib_parse,
 )
 
+
 class MofosexIE(InfoExtractor):
     _VALID_URL = r'^(?:https?://)?(?:www\.)?(?P<url>mofosex\.com/videos/(?P<videoid>[0-9]+)/.*?\.html)'
     _TEST = {

youtube_dl/extractor/mtv.py

@@ -18,6 +18,7 @@ def _media_xml_tag(tag):
 
 
 class MTVServicesInfoExtractor(InfoExtractor):
+
     @staticmethod
     def _id_from_uri(uri):
         return uri.split(':')[-1]
@@ -173,7 +174,7 @@ class MTVIE(MTVServicesInfoExtractor):
             m_vevo = re.search(r'isVevoVideo = true;.*?vevoVideoId = "(.*?)";',
                                webpage, re.DOTALL)
             if m_vevo:
-                vevo_id = m_vevo.group(1);
+                vevo_id = m_vevo.group(1)
                 self.to_screen('Vevo video detected: %s' % vevo_id)
                 return self.url_result('vevo:%s' % vevo_id, ie='Vevo')
 

youtube_dl/extractor/muzu.py

@@ -37,7 +37,7 @@ class MuzuTVIE(InfoExtractor):
         player_info_page = self._download_webpage('http://player.muzu.tv/player/playerInit?ai=%s' % video_id,
                                                   video_id, u'Downloading player info')
         video_info = json.loads(player_info_page)['videos'][0]
-        for quality in ['1080' , '720', '480', '360']:
+        for quality in ['1080', '720', '480', '360']:
             if video_info.get('v%s' % quality):
                 break
 

youtube_dl/extractor/myvideo.py

@@ -14,8 +14,8 @@ from ..utils import (
 )
 
 
-
 class MyVideoIE(InfoExtractor):
+
     """Information Extractor for myvideo.de."""
 
     _VALID_URL = r'(?:http://)?(?:www\.)?myvideo\.de/(?:[^/]+/)?watch/([0-9]+)/([^?/]+).*'
@@ -32,7 +32,7 @@ class MyVideoIE(InfoExtractor):
     # Original Code from: https://github.com/dersphere/plugin.video.myvideo_de.git
     # Released into the Public Domain by Tristan Fischer on 2013-05-19
     # https://github.com/rg3/youtube-dl/pull/842
-    def __rc4crypt(self,data, key):
+    def __rc4crypt(self, data, key):
         x = 0
         box = list(range(256))
         for i in list(range(256)):
@@ -48,10 +48,10 @@ class MyVideoIE(InfoExtractor):
             out += chr(compat_ord(char) ^ box[(box[x] + box[y]) % 256])
         return out
 
-    def __md5(self,s):
+    def __md5(self, s):
         return hashlib.md5(s).hexdigest().encode()
 
-    def _real_extract(self,url):
+    def _real_extract(self, url):
         mobj = re.match(self._VALID_URL, url)
         if mobj is None:
             raise ExtractorError(u'invalid URL: %s' % url)
@@ -187,4 +187,3 @@ class MyVideoIE(InfoExtractor):
             'video_hls_playlist': video_hls_playlist,
             'player_url': video_swfobj,
         }]
-

youtube_dl/extractor/naver.py

@@ -31,7 +31,7 @@ class NaverIE(InfoExtractor):
             raise ExtractorError(u'couldn\'t extract vid and key')
         vid = m_id.group(1)
         key = m_id.group(2)
-        query = compat_urllib_parse.urlencode({'vid': vid, 'inKey': key,})
+        query = compat_urllib_parse.urlencode({'vid': vid, 'inKey': key, })
         query_urls = compat_urllib_parse.urlencode({
             'masterVid': vid,
             'protocol': 'p2p',

youtube_dl/extractor/nhl.py

@@ -11,6 +11,7 @@ from ..utils import (
 
 
 class NHLBaseInfoExtractor(InfoExtractor):
+
     @staticmethod
     def _fix_json(json_string):
         return json_string.replace('\\\'', '\'')

youtube_dl/extractor/niconico.py

@@ -61,7 +61,7 @@ class NiconicoIE(InfoExtractor):
         }
         # Convert to UTF-8 *before* urlencode because Python 2.x's urlencode
         # chokes on unicode
-        login_form = dict((k.encode('utf-8'), v.encode('utf-8')) for k,v in login_form_strs.items())
+        login_form = dict((k.encode('utf-8'), v.encode('utf-8')) for k, v in login_form_strs.items())
         login_data = compat_urllib_parse.urlencode(login_form).encode('utf-8')
         request = compat_urllib_request.Request(
             u'https://secure.nicovideo.jp/secure/login', login_data)

youtube_dl/extractor/normalboots.py

@@ -7,6 +7,7 @@ from ..utils import (
     unified_strdate,
 )
 
+
 class NormalbootsIE(InfoExtractor):
     _VALID_URL = r'(?:http://)?(?:www\.)?normalboots\.com/video/(?P<videoid>[0-9a-z-]*)/?$'
     _TEST = {

youtube_dl/extractor/ooyala.py

@@ -4,6 +4,7 @@ import json
 from .common import InfoExtractor
 from ..utils import unescapeHTML
 
+
 class OoyalaIE(InfoExtractor):
     _VALID_URL = r'https?://.+?\.ooyala\.com/.*?(?:embedCode|ec)=(?P<id>.+?)(&|$)'
 
@@ -46,10 +47,10 @@ class OoyalaIE(InfoExtractor):
         mobile_player = self._download_webpage(mobile_url, embedCode)
         videos_info = self._search_regex(
             r'var streams=window.oo_testEnv\?\[\]:eval\("\((\[{.*?}\])\)"\);',
-            mobile_player, u'info').replace('\\"','"')
+            mobile_player, u'info').replace('\\"', '"')
-        videos_more_info = self._search_regex(r'eval\("\(({.*?\\"promo\\".*?})\)"', mobile_player, u'more info').replace('\\"','"')
+        videos_more_info = self._search_regex(r'eval\("\(({.*?\\"promo\\".*?})\)"', mobile_player, u'more info').replace('\\"', '"')
         videos_info = json.loads(videos_info)
-        videos_more_info =json.loads(videos_more_info)
+        videos_more_info = json.loads(videos_more_info)
 
         if videos_more_info.get('lineup'):
             videos = [self._extract_result(info, more_info) for (info, more_info) in zip(videos_info, videos_more_info['lineup'])]
@@ -60,4 +61,3 @@ class OoyalaIE(InfoExtractor):
                     }
         else:
             return self._extract_result(videos_info[0], videos_more_info)
-        

youtube_dl/extractor/photobucket.py

@@ -8,7 +8,9 @@ from ..utils import (
     ExtractorError,
 )
 
+
 class PhotobucketIE(InfoExtractor):
+
     """Information extractor for photobucket.com."""
 
     # TODO: the original _VALID_URL was:

youtube_dl/extractor/pornhub.py

@@ -42,7 +42,7 @@ class PornHubIE(InfoExtractor):
         if thumbnail:
             thumbnail = compat_urllib_parse.unquote(thumbnail)
 
-        video_urls = list(map(compat_urllib_parse.unquote , re.findall(r'"quality_[0-9]{3}p":"([^"]+)', webpage)))
+        video_urls = list(map(compat_urllib_parse.unquote, re.findall(r'"quality_[0-9]{3}p":"([^"]+)', webpage)))
         if webpage.find('"encrypted":true') != -1:
             password = self._html_search_regex(r'"video_title":"([^"]+)', webpage, 'password').replace('+', ' ')
             video_urls = list(map(lambda s: aes_decrypt_text(s, password, 32).decode('utf-8'), video_urls))

youtube_dl/extractor/pornotube.py

@@ -35,10 +35,11 @@ class PornotubeIE(InfoExtractor):
         video_url = self._search_regex(VIDEO_URL_RE, webpage, u'video url')
         video_url = compat_urllib_parse.unquote(video_url)
 
-        #Get the uploaded date
+        # Get the uploaded date
         VIDEO_UPLOADED_RE = r'<div class="video_added_by">Added (?P<date>[0-9\/]+) by'
         upload_date = self._html_search_regex(VIDEO_UPLOADED_RE, webpage, u'upload date', fatal=False)
-        if upload_date: upload_date = unified_strdate(upload_date)
+        if upload_date:
+            upload_date = unified_strdate(upload_date)
         age_limit = self._rta_search(webpage)
 
         info = {'id': video_id,

youtube_dl/extractor/ringtv.py

@@ -41,4 +41,3 @@ class RingTVIE(InfoExtractor):
             'thumbnail': thumbnail_url,
             'description': description,
         }
-

youtube_dl/extractor/rtlnow.py

@@ -12,6 +12,7 @@ from ..utils import (
 
 
 class RTLnowIE(InfoExtractor):
+
     """Information Extractor for RTL NOW, RTL2 NOW, RTL NITRO, SUPER RTL NOW, VOX NOW and n-tv NOW"""
     _VALID_URL = r'(?:http://)?(?P<url>(?P<domain>rtl-now\.rtl\.de|rtl2now\.rtl2\.de|(?:www\.)?voxnow\.de|(?:www\.)?rtlnitronow\.de|(?:www\.)?superrtlnow\.de|(?:www\.)?n-tvnow\.de)/+[a-zA-Z0-9-]+/[a-zA-Z0-9-]+\.php\?(?:container_id|film_id)=(?P<video_id>[0-9]+)&player=1(?:&season=[0-9]+)?(?:&.*)?)'
     _TESTS = [{

youtube_dl/extractor/servingsys.py

@@ -67,5 +67,3 @@ class ServingSysIE(InfoExtractor):
             'title': title,
             'entries': entries,
         }
-
- 

youtube_dl/extractor/smotri.py

@@ -202,15 +202,15 @@ class SmotriIE(InfoExtractor):
 
         video_uploader = self._html_search_regex(
             '<div class="DescrUser"><div>Автор.*?onmouseover="popup_user_info[^"]+">(.*?)</a>',
-            video_page, 'uploader', fatal=False, flags=re.MULTILINE|re.DOTALL)
+            video_page, 'uploader', fatal=False, flags=re.MULTILINE | re.DOTALL)
 
         video_uploader_id = self._html_search_regex(
             '<div class="DescrUser"><div>Автор.*?onmouseover="popup_user_info\\(.*?\'([^\']+)\'\\);">',
-            video_page, 'uploader id', fatal=False, flags=re.MULTILINE|re.DOTALL)
+            video_page, 'uploader id', fatal=False, flags=re.MULTILINE | re.DOTALL)
 
         video_view_count = self._html_search_regex(
             'Общее количество просмотров.*?<span class="Number">(\\d+)</span>',
-            video_page, 'view count', fatal=False, flags=re.MULTILINE|re.DOTALL)
+            video_page, 'view count', fatal=False, flags=re.MULTILINE | re.DOTALL)
 
         return {
             'id': video_id,

youtube_dl/extractor/sohu.py

@@ -69,7 +69,7 @@ class SohuIE(InfoExtractor):
                         (allot, prot, clipsURL[i], su[i]))
             part_str = self._download_webpage(
                 part_url, video_id,
-                note=u'Downloading part %d of %d' % (i+1, part_count))
+                note=u'Downloading part %d of %d' % (i + 1, part_count))
 
             part_info = part_str.split('|')
             video_url = '%s%s?key=%s' % (part_info[0], su[i], part_info[3])

youtube_dl/extractor/soundcloud.py

@@ -17,6 +17,7 @@ from ..utils import (
 
 
 class SoundcloudIE(InfoExtractor):
+
     """Information extractor for soundcloud.com
        To access the media, the uid of the song and a stream token
        must be extracted from the page source and the script must make
@@ -216,6 +217,7 @@ class SoundcloudIE(InfoExtractor):
         info = json.loads(info_json)
         return self._extract_info_dict(info, full_title, secret_token=token)
 
+
 class SoundcloudSetIE(SoundcloudIE):
     _VALID_URL = r'^(?:https?://)?(?:www\.)?soundcloud\.com/([\w\d-]+)/sets/([\w\d-]+)(?:[?].*)?$'
     IE_NAME = 'soundcloud:set'
@@ -272,12 +274,12 @@ class SoundcloudUserIE(SoundcloudIE):
 
         tracks = []
         for i in itertools.count():
-            data = compat_urllib_parse.urlencode({'offset': i*50,
+            data = compat_urllib_parse.urlencode({'offset': i * 50,
                                                   'client_id': self._CLIENT_ID,
                                                   })
             tracks_url = 'http://api.soundcloud.com/users/%s/tracks.json?' % user['id'] + data
             response = self._download_webpage(tracks_url, uploader,
-                'Downloading tracks page %s' % (i+1))
+                                              'Downloading tracks page %s' % (i + 1))
             new_tracks = json.loads(response)
             tracks.extend(self._extract_info_dict(track, quiet=True) for track in new_tracks)
             if len(new_tracks) < 50:

youtube_dl/extractor/spankwire.py

@@ -45,7 +45,7 @@ class SpankwireIE(InfoExtractor):
         description = self._html_search_regex(
             r'<div\s+id="descriptionContent">([^<]+)<', webpage, 'description', fatal=False)
 
-        video_urls = list(map(compat_urllib_parse.unquote , re.findall(r'flashvars\.quality_[0-9]{3}p = "([^"]+)', webpage)))
+        video_urls = list(map(compat_urllib_parse.unquote, re.findall(r'flashvars\.quality_[0-9]{3}p = "([^"]+)', webpage)))
         if webpage.find('flashvars\.encrypted = "true"') != -1:
             password = self._html_search_regex(r'flashvars\.video_title = "([^"]+)', webpage, 'password').replace('+', ' ')
             video_urls = list(map(lambda s: aes_decrypt_text(s, password, 32).decode('utf-8'), video_urls))

youtube_dl/extractor/steam.py

@@ -38,7 +38,6 @@ class SteamIE(InfoExtractor):
         ]
     }
 
-
     @classmethod
     def suitable(cls, url):
         """Receives a URL and returns True if suitable for this IE."""
@@ -67,7 +66,7 @@ class SteamIE(InfoExtractor):
         thumbsRE = r'<img class="movie_thumb" src="(?P<thumbnail>.+?)">'
         thumbs = re.finditer(thumbsRE, webpage)
         videos = []
-        for vid,vtitle,thumb in zip(mweb,titles,thumbs):
+        for vid, vtitle, thumb in zip(mweb, titles, thumbs):
             video_id = vid.group('videoID')
             title = vtitle.group('videoName')
             video_url = vid.group('videoURL')
@@ -75,8 +74,8 @@ class SteamIE(InfoExtractor):
             if not video_url:
                 raise ExtractorError(u'Cannot find video url for %s' % video_id)
             info = {
-                'id':video_id,
+                'id': video_id,
-                'url':video_url,
+                'url': video_url,
                 'ext': 'flv',
                 'title': unescapeHTML(title),
                 'thumbnail': video_thumb

youtube_dl/extractor/subtitles.py

@@ -7,6 +7,7 @@ from ..utils import (
 
 
 class SubtitlesInfoExtractor(InfoExtractor):
+
     @property
     def _have_to_download_any_subtitles(self):
         return any([self._downloader.params.get('writesubtitles', False),

youtube_dl/extractor/ted.py

@@ -11,7 +11,7 @@ from ..utils import (
 
 
 class TEDIE(SubtitlesInfoExtractor):
-    _VALID_URL=r'''http://www\.ted\.com/
+    _VALID_URL = r'''http://www\.ted\.com/
                    (
                         ((?P<type_playlist>playlists)/(?P<playlist_id>\d+)) # We have a playlist
                         |
@@ -36,15 +36,14 @@ class TEDIE(SubtitlesInfoExtractor):
         return re.match(cls._VALID_URL, url, re.VERBOSE) is not None
 
     def _real_extract(self, url):
-        m=re.match(self._VALID_URL, url, re.VERBOSE)
+        m = re.match(self._VALID_URL, url, re.VERBOSE)
         if m.group('type_talk'):
             return self._talk_info(url)
-        else :
+        else:
-            playlist_id=m.group('playlist_id')
+            playlist_id = m.group('playlist_id')
-            name=m.group('name')
+            name = m.group('name')
-            self.to_screen(u'Getting info of playlist %s: "%s"' % (playlist_id,name))
+            self.to_screen(u'Getting info of playlist %s: "%s"' % (playlist_id, name))
-            return [self._playlist_videos_info(url,name,playlist_id)]
+            return [self._playlist_videos_info(url, name, playlist_id)]
-
 
     def _playlist_videos_info(self, url, name, playlist_id):
         '''Returns the videos of the playlist'''
@@ -67,7 +66,7 @@ class TEDIE(SubtitlesInfoExtractor):
 
     def _talk_info(self, url, video_id=0):
         """Return the video for the talk in the url"""
-        m = re.match(self._VALID_URL, url,re.VERBOSE)
+        m = re.match(self._VALID_URL, url, re.VERBOSE)
         video_name = m.group('name')
         webpage = self._download_webpage(url, video_id, 'Downloading \"%s\" page' % video_name)
         self.report_extraction(video_name)
@@ -78,7 +77,7 @@ class TEDIE(SubtitlesInfoExtractor):
                                        webpage, 'json data')
         info = json.loads(json_data)
         desc = self._html_search_regex(r'<div class="talk-intro">.*?<p.*?>(.*?)</p>',
-                                       webpage, 'description', flags = re.DOTALL)
+                                       webpage, 'description', flags=re.DOTALL)
 
         thumbnail = self._search_regex(r'</span>[\s.]*</div>[\s.]*<img src="(.*?)"',
                                        webpage, 'thumbnail')

youtube_dl/extractor/testurl.py

@@ -7,6 +7,7 @@ from ..utils import ExtractorError
 
 
 class TestURLIE(InfoExtractor):
+
     """ Allows adressing of the test cases as test:yout.*be_1 """
 
     IE_DESC = False  # Do not list

youtube_dl/extractor/tf1.py

@@ -5,7 +5,9 @@ import re
 
 from .common import InfoExtractor
 
+
 class TF1IE(InfoExtractor):
+
     """TF1 uses the wat.tv player."""
     _VALID_URL = r'http://videos\.tf1\.fr/.*-(.*?)\.html'
     _TEST = {

youtube_dl/extractor/theplatform.py

@@ -86,14 +86,14 @@ class ThePlatformIE(InfoExtractor):
             'formats': formats,
             'description': info['description'],
             'thumbnail': info['defaultThumbnailUrl'],
-            'duration': info['duration']//1000,
+            'duration': info['duration'] // 1000,
         }
 
     def _real_extract(self, url):
         mobj = re.match(self._VALID_URL, url)
         video_id = mobj.group('id')
         if mobj.group('config'):
-            config_url = url+ '&form=json'
+            config_url = url + '&form=json'
             config_url = config_url.replace('swf/', 'config/')
             config_json = self._download_webpage(config_url, video_id, u'Downloading config')
             config = json.loads(config_json)

youtube_dl/extractor/thisav.py

@@ -1,4 +1,4 @@
-#coding: utf-8
+# coding: utf-8
 from __future__ import unicode_literals
 
 import re