aboutsummaryrefslogtreecommitdiff
path: root/youtube_dl/extractor/brightcove.py
diff options
context:
space:
mode:
authorYen Chi Hsuan <yan12125@gmail.com>2015-04-16 00:27:39 +0800
committerYen Chi Hsuan <yan12125@gmail.com>2015-04-16 00:27:39 +0800
commit9fc03aa87c74cb199f9e1e146ac5cb71b166d3e7 (patch)
treee9b276896d1287690228d9dab35c469c6572dd44 /youtube_dl/extractor/brightcove.py
parentc798f15b989bc8c3578c5b0baf75f4fb4760ba81 (diff)
downloadyoutube-dl-9fc03aa87c74cb199f9e1e146ac5cb71b166d3e7.tar.xz
[brightcove] Always return lists from _extract_brightcove_urls
In Python 3, filter() returns an iterable object, which is equivalently to True even for an empty result set. It causes false positive playlists in generic extraction logic.
Diffstat (limited to 'youtube_dl/extractor/brightcove.py')
-rw-r--r--youtube_dl/extractor/brightcove.py2
1 files changed, 1 insertions, 1 deletions
diff --git a/youtube_dl/extractor/brightcove.py b/youtube_dl/extractor/brightcove.py
index 117cb00e6..4f60d5366 100644
--- a/youtube_dl/extractor/brightcove.py
+++ b/youtube_dl/extractor/brightcove.py
@@ -188,7 +188,7 @@ class BrightcoveIE(InfoExtractor):
[^>]*?>\s*<param\s+name="movie"\s+value="https?://[^/]*brightcove\.com/
).+?>\s*</object>''',
webpage)
- return filter(None, [cls._build_brighcove_url(m) for m in matches])
+ return list(filter(None, [cls._build_brighcove_url(m) for m in matches]))
def _real_extract(self, url):
url, smuggled_data = unsmuggle_url(url, {})