diff options
author | remitamine <remitamine@gmail.com> | 2015-12-05 21:14:43 +0100 |
---|---|---|
committer | remitamine <remitamine@gmail.com> | 2015-12-28 18:45:01 +0100 |
commit | d497a201cab0eb588e9b7f75494f5494c9663e67 (patch) | |
tree | a85184797f9d073590f7e52f9ae944a980320214 /youtube_dl/extractor | |
parent | 51c4fec0d58af709a06bd75338b23eace412362f (diff) |
[common] use specific variable for protocol preference in _sort_formats
Diffstat (limited to 'youtube_dl/extractor')
-rw-r--r-- | youtube_dl/extractor/common.py | 11 |
1 files changed, 5 insertions, 6 deletions
diff --git a/youtube_dl/extractor/common.py b/youtube_dl/extractor/common.py index 6ab2d68d6..76f5b8b05 100644 --- a/youtube_dl/extractor/common.py +++ b/youtube_dl/extractor/common.py @@ -18,7 +18,6 @@ from ..compat import ( compat_http_client, compat_urllib_error, compat_urllib_parse, - compat_urllib_parse_urlparse, compat_urlparse, compat_str, compat_etree_fromstring, @@ -42,6 +41,7 @@ from ..utils import ( url_basename, xpath_text, xpath_with_ns, + determine_protocol, ) @@ -776,14 +776,12 @@ class InfoExtractor(object): preference = f.get('preference') if preference is None: - proto = f.get('protocol') - if proto is None: - proto = compat_urllib_parse_urlparse(f.get('url', '')).scheme - - preference = 0 if proto in ['http', 'https'] else -0.1 + preference = 0 if f.get('ext') in ['f4f', 'f4m']: # Not yet supported preference -= 0.5 + proto_preference = 0 if determine_protocol(f) in ['http', 'https'] else -0.1 + if f.get('vcodec') == 'none': # audio only if self._downloader.params.get('prefer_free_formats'): ORDER = ['aac', 'mp3', 'm4a', 'webm', 'ogg', 'opus'] @@ -814,6 +812,7 @@ class InfoExtractor(object): f.get('vbr') if f.get('vbr') is not None else -1, f.get('height') if f.get('height') is not None else -1, f.get('width') if f.get('width') is not None else -1, + proto_preference, ext_preference, f.get('abr') if f.get('abr') is not None else -1, audio_ext_preference, |