diff options
| author | Yen Chi Hsuan <yan12125@gmail.com> | 2015-04-16 00:27:39 +0800 |
|---|---|---|
| committer | Yen Chi Hsuan <yan12125@gmail.com> | 2015-04-16 00:27:39 +0800 |
| commit | 9fc03aa87c74cb199f9e1e146ac5cb71b166d3e7 (patch) | |
| tree | e9b276896d1287690228d9dab35c469c6572dd44 /youtube_dl/extractor | |
| parent | c798f15b989bc8c3578c5b0baf75f4fb4760ba81 (diff) | |
[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')
| -rw-r--r-- | youtube_dl/extractor/brightcove.py | 2 |
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, {}) |