diff options
| -rw-r--r-- | devscripts/check-porn.py | 8 | ||||
| -rw-r--r-- | test/test_swfinterp.py | 4 | ||||
| -rwxr-xr-x | youtube_dl/YoutubeDL.py | 8 | ||||
| -rw-r--r-- | youtube_dl/__init__.py | 16 | ||||
| -rw-r--r-- | youtube_dl/downloader/common.py | 10 | ||||
| -rw-r--r-- | youtube_dl/downloader/f4m.py | 4 | ||||
| -rw-r--r-- | youtube_dl/extractor/adobetv.py | 7 | ||||
| -rw-r--r-- | youtube_dl/extractor/common.py | 4 | ||||
| -rw-r--r-- | youtube_dl/extractor/defense.py | 5 | ||||
| -rw-r--r-- | youtube_dl/utils.py | 4 | 
10 files changed, 36 insertions, 34 deletions
diff --git a/devscripts/check-porn.py b/devscripts/check-porn.py index 216282712..6a5bd9eda 100644 --- a/devscripts/check-porn.py +++ b/devscripts/check-porn.py @@ -45,12 +45,12 @@ for test in get_testcases():          RESULT = ('.' + domain + '\n' in LIST or '\n' + domain + '\n' in LIST) -    if RESULT and ('info_dict' not in test or 'age_limit' not in test['info_dict'] -                   or test['info_dict']['age_limit'] != 18): +    if RESULT and ('info_dict' not in test or 'age_limit' not in test['info_dict'] or +                   test['info_dict']['age_limit'] != 18):          print('\nPotential missing age_limit check: {0}'.format(test['name'])) -    elif not RESULT and ('info_dict' in test and 'age_limit' in test['info_dict'] -                         and test['info_dict']['age_limit'] == 18): +    elif not RESULT and ('info_dict' in test and 'age_limit' in test['info_dict'] and +                         test['info_dict']['age_limit'] == 18):          print('\nPotential false negative: {0}'.format(test['name']))      else: diff --git a/test/test_swfinterp.py b/test/test_swfinterp.py index 9f18055e6..f1e899819 100644 --- a/test/test_swfinterp.py +++ b/test/test_swfinterp.py @@ -34,8 +34,8 @@ def _make_testfunc(testfile):      def test_func(self):          as_file = os.path.join(TEST_DIR, testfile)          swf_file = os.path.join(TEST_DIR, test_id + '.swf') -        if ((not os.path.exists(swf_file)) -                or os.path.getmtime(swf_file) < os.path.getmtime(as_file)): +        if ((not os.path.exists(swf_file)) or +                os.path.getmtime(swf_file) < os.path.getmtime(as_file)):              # Recompile              try:                  subprocess.check_call([ diff --git a/youtube_dl/YoutubeDL.py b/youtube_dl/YoutubeDL.py index 88809783b..ca7c3f5c6 100755 --- a/youtube_dl/YoutubeDL.py +++ b/youtube_dl/YoutubeDL.py @@ -308,8 +308,8 @@ class YoutubeDL(object):                      raise          if (sys.version_info >= (3,) and sys.platform != 'win32' and -                sys.getfilesystemencoding() in ['ascii', 'ANSI_X3.4-1968'] -                and not params.get('restrictfilenames', False)): +                sys.getfilesystemencoding() in ['ascii', 'ANSI_X3.4-1968'] and +                not params.get('restrictfilenames', False)):              # On Python 3, the Unicode filesystem API will throw errors (#1474)              self.report_warning(                  'Assuming --restrict-filenames since file system encoding ' @@ -1366,8 +1366,8 @@ class YoutubeDL(object):          """Download a given list of URLs."""          outtmpl = self.params.get('outtmpl', DEFAULT_OUTTMPL)          if (len(url_list) > 1 and -                '%' not in outtmpl -                and self.params.get('max_downloads') != 1): +                '%' not in outtmpl and +                self.params.get('max_downloads') != 1):              raise SameFileError(outtmpl)          for url in url_list: diff --git a/youtube_dl/__init__.py b/youtube_dl/__init__.py index eac2a26ec..25ab3fdfe 100644 --- a/youtube_dl/__init__.py +++ b/youtube_dl/__init__.py @@ -189,14 +189,14 @@ def _real_main(argv=None):          # 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) -               or (opts.format == '-1' and opts.usetitle and '%(title)s-%(id)s-%(format)s.%(ext)s') -               or (opts.format == '-1' and '%(id)s-%(format)s.%(ext)s') -               or (opts.usetitle and opts.autonumber and '%(autonumber)s-%(title)s-%(id)s.%(ext)s') -               or (opts.usetitle and '%(title)s-%(id)s.%(ext)s') -               or (opts.useid and '%(id)s.%(ext)s') -               or (opts.autonumber and '%(autonumber)s-%(id)s.%(ext)s') -               or DEFAULT_OUTTMPL) +    outtmpl = ((opts.outtmpl is not None and opts.outtmpl) or +               (opts.format == '-1' and opts.usetitle and '%(title)s-%(id)s-%(format)s.%(ext)s') or +               (opts.format == '-1' and '%(id)s-%(format)s.%(ext)s') or +               (opts.usetitle and opts.autonumber and '%(autonumber)s-%(title)s-%(id)s.%(ext)s') or +               (opts.usetitle and '%(title)s-%(id)s.%(ext)s') or +               (opts.useid and '%(id)s.%(ext)s') or +               (opts.autonumber and '%(autonumber)s-%(id)s.%(ext)s') or +               DEFAULT_OUTTMPL)      if not os.path.splitext(outtmpl)[1] and opts.extractaudio:          parser.error('Cannot download a video and extract audio into the same'                       ' file! Use "{0}.%(ext)s" instead of "{0}" as the output' diff --git a/youtube_dl/downloader/common.py b/youtube_dl/downloader/common.py index 45e55b99c..3ae90021a 100644 --- a/youtube_dl/downloader/common.py +++ b/youtube_dl/downloader/common.py @@ -311,14 +311,14 @@ class FileDownloader(object):          """          nooverwrites_and_exists = ( -            self.params.get('nooverwrites', False) -            and os.path.exists(encodeFilename(filename)) +            self.params.get('nooverwrites', False) and +            os.path.exists(encodeFilename(filename))          )          continuedl_and_exists = ( -            self.params.get('continuedl', False) -            and os.path.isfile(encodeFilename(filename)) -            and not self.params.get('nopart', False) +            self.params.get('continuedl', False) and +            os.path.isfile(encodeFilename(filename)) and +            not self.params.get('nopart', False)          )          # Check file already present diff --git a/youtube_dl/downloader/f4m.py b/youtube_dl/downloader/f4m.py index b40ebfa50..7b8fe8cf5 100644 --- a/youtube_dl/downloader/f4m.py +++ b/youtube_dl/downloader/f4m.py @@ -325,8 +325,8 @@ class F4mFD(FileDownloader):                  state['frag_index'] += 1              estimated_size = ( -                (state['downloaded_bytes'] + frag_total_bytes) -                / (state['frag_index'] + 1) * total_frags) +                (state['downloaded_bytes'] + frag_total_bytes) / +                (state['frag_index'] + 1) * total_frags)              time_now = time.time()              state['total_bytes_estimate'] = estimated_size              state['elapsed'] = time_now - start diff --git a/youtube_dl/extractor/adobetv.py b/youtube_dl/extractor/adobetv.py index 28e07f8b0..97d128560 100644 --- a/youtube_dl/extractor/adobetv.py +++ b/youtube_dl/extractor/adobetv.py @@ -28,7 +28,6 @@ class AdobeTVIE(InfoExtractor):      def _real_extract(self, url):          video_id = self._match_id(url) -          webpage = self._download_webpage(url, video_id)          player = self._parse_json( @@ -44,8 +43,10 @@ class AdobeTVIE(InfoExtractor):              self._html_search_meta('datepublished', webpage, 'upload date'))          duration = parse_duration( -            self._html_search_meta('duration', webpage, 'duration') -            or self._search_regex(r'Runtime:\s*(\d{2}:\d{2}:\d{2})', webpage, 'duration')) +            self._html_search_meta('duration', webpage, 'duration') or +            self._search_regex( +                r'Runtime:\s*(\d{2}:\d{2}:\d{2})', +                webpage, 'duration', fatal=False))          view_count = str_to_int(self._search_regex(              r'<div class="views">\s*Views?:\s*([\d,.]+)\s*</div>', diff --git a/youtube_dl/extractor/common.py b/youtube_dl/extractor/common.py index ee64ad329..d3f86cf4a 100644 --- a/youtube_dl/extractor/common.py +++ b/youtube_dl/extractor/common.py @@ -808,8 +808,8 @@ class InfoExtractor(object):              media_nodes = manifest.findall('{http://ns.adobe.com/f4m/2.0}media')          for i, media_el in enumerate(media_nodes):              if manifest_version == '2.0': -                manifest_url = ('/'.join(manifest_url.split('/')[:-1]) + '/' -                                + (media_el.attrib.get('href') or media_el.attrib.get('url'))) +                manifest_url = ('/'.join(manifest_url.split('/')[:-1]) + '/' + +                                (media_el.attrib.get('href') or media_el.attrib.get('url')))              tbr = int_or_none(media_el.attrib.get('bitrate'))              formats.append({                  'format_id': '-'.join(filter(None, [f4m_id, 'f4m-%d' % (i if tbr is None else tbr)])), diff --git a/youtube_dl/extractor/defense.py b/youtube_dl/extractor/defense.py index 2b90bf4fc..98e3aedfd 100644 --- a/youtube_dl/extractor/defense.py +++ b/youtube_dl/extractor/defense.py @@ -25,8 +25,9 @@ class DefenseGouvFrIE(InfoExtractor):              r"flashvars.pvg_id=\"(\d+)\";",              webpage, 'ID') -        json_url = ('http://static.videos.gouv.fr/brightcovehub/export/json/' -                    + video_id) +        json_url = ( +            'http://static.videos.gouv.fr/brightcovehub/export/json/%s' % +            video_id)          info = self._download_json(json_url, title, 'Downloading JSON config')          video_url = info['renditions'][0]['url'] diff --git a/youtube_dl/utils.py b/youtube_dl/utils.py index 238b6556b..475fad3c9 100644 --- a/youtube_dl/utils.py +++ b/youtube_dl/utils.py @@ -900,8 +900,8 @@ def _windows_write_string(s, out):      def not_a_console(handle):          if handle == INVALID_HANDLE_VALUE or handle is None:              return True -        return ((GetFileType(handle) & ~FILE_TYPE_REMOTE) != FILE_TYPE_CHAR -                or GetConsoleMode(handle, ctypes.byref(ctypes.wintypes.DWORD())) == 0) +        return ((GetFileType(handle) & ~FILE_TYPE_REMOTE) != FILE_TYPE_CHAR or +                GetConsoleMode(handle, ctypes.byref(ctypes.wintypes.DWORD())) == 0)      if not_a_console(h):          return False  | 
