From: Jaime Marquínez Ferrándiz Date: Fri, 27 Sep 2013 06:08:43 +0000 (+0200) Subject: Don't embed subtitles if the list is empty or the field is not set (fixes #1510) X-Git-Url: http://git.bitcoin.ninja/?a=commitdiff_plain;h=74bab3f0a4b601a7618f279afbd352bbc51dc3ce;p=youtube-dl Don't embed subtitles if the list is empty or the field is not set (fixes #1510) --- diff --git a/youtube_dl/PostProcessor.py b/youtube_dl/PostProcessor.py index ae56d2082..3ee1d3c58 100644 --- a/youtube_dl/PostProcessor.py +++ b/youtube_dl/PostProcessor.py @@ -444,8 +444,11 @@ class FFmpegEmbedSubtitlePP(FFmpegPostProcessor): if information['ext'] != u'mp4': self._downloader.to_screen(u'[ffmpeg] Subtitles can only be embedded in mp4 files') return True, information - sub_langs = [key for key in information['subtitles']] + if not information.get('subtitles'): + self._downloader.to_screen(u'[ffmpeg] There aren\'t any subtitles to embed') + return True, information + sub_langs = [key for key in information['subtitles']] filename = information['filepath'] input_files = [filename] + [subtitles_filename(filename, lang, self._subformat) for lang in sub_langs]