diff options
| author | wm4 <wm4@nowhere> | 2013-08-15 23:24:53 +0200 |
|---|---|---|
| committer | wm4 <wm4@nowhere> | 2013-08-15 23:40:04 +0200 |
| commit | 8b245c4d4b644e404b2e776b6c0ea42d27fa2005 (patch) | |
| tree | 754e88b2fcdae1fc97a7b17dc33871f6adf6127e /sub | |
| parent | 3e6ed769357aa317bab5866da39cf0704e62fd5e (diff) | |
| download | mpv-8b245c4d4b644e404b2e776b6c0ea42d27fa2005.tar.bz2 mpv-8b245c4d4b644e404b2e776b6c0ea42d27fa2005.tar.xz | |
sd_lavc_conv: don't check AV_CODEC_PROP_TEXT_SUB flag
Not actually useful. This would break whenever a new text subtitle
format would be added, which requires a binary->text transformation.
(mov_text is one such format; disable it.) In general, we would have
to know which packet formats are binary, which we don't, so the only
reasonable way to handle this is a white list.
Diffstat (limited to 'sub')
| -rw-r--r-- | sub/sd_lavc_conv.c | 10 |
1 files changed, 3 insertions, 7 deletions
diff --git a/sub/sd_lavc_conv.c b/sub/sd_lavc_conv.c index 0c8160495c..ee783f08a4 100644 --- a/sub/sd_lavc_conv.c +++ b/sub/sd_lavc_conv.c @@ -40,20 +40,16 @@ static bool supports_format(const char *format) const AVCodecDescriptor *desc = avcodec_descriptor_get(cid); if (!desc) return false; -#if HAVE_AV_CODEC_PROP_TEXT_SUB - // These are documented to support AVSubtitleRect->ass. - return desc->props & AV_CODEC_PROP_TEXT_SUB; -#else + // These are known to support AVSubtitleRect->ass. const char *whitelist[] = - {"text", "ass", "ssa", "mov_text", "srt", "subrip", "microdvd", "mpl2", - "jacosub", "pjs", "sami", "realtext", "subviewer", "subviewer1", + {"text", "ass", "ssa", "srt", "subrip", "microdvd", "mpl2", + "jacosub", "pjs", "sami", "realtext", "subviewer", "subviewer1", "vplayer", "webvtt", 0}; for (int n = 0; whitelist[n]; n++) { if (strcmp(format, whitelist[n]) == 0) return true; } return false; -#endif } // Disable style definitions generated by the libavcodec converter. |