[MPlayer-dev-eng] [PATCH] Handle AV_CODEC_ID_SUBRIP subtitles

[Philip Langdale]

Hi all,

For anyone who's following ffmpeg developments, I'm about to
commit a change that will cause matroskadec to mark srt subtitles
as CODEC_ID_SUBRIP, rather than CODEC_ID_TEXT. This reflects the
fact that the subtitles use Subrip markup, and are not plain text
in the strict definition. Within ffmpeg, this will enable proper
transcoding of these subtitles.

Today, mplayer explicitly handles CODEC_ID_TEXT in such a way that
the Subrip markup is parses - which is great, but it also means
that we need to explicitly add support for CODEC_ID_SUBRIP, which
the attached patch does.

I have verified this change with my pending ffmpeg changes, but I
will not be committing those to ffmpeg until after this goes in.

Thanks,

--phil
-------------- next part --------------
A non-text attachment was scrubbed...
Name: subrip.diff
Type: text/x-patch
Size: 1745 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20120818/c1e2229f/attachment-0001.bin>