[FFmpeg-devel] [PATCH] matroska subtitle tracks support
Tue Jul 10 11:41:46 CEST 2007
Michael Niedermayer wrote:
> On Mon, Jul 09, 2007 at 11:11:59PM +0200, Aurelien Jacobs wrote:
>> On Mon, 9 Jul 2007 18:22:18 +0200
>> Michael Niedermayer <michaelni at gmx.at> wrote:
>> > Hi
>> > On Mon, Jul 09, 2007 at 03:47:21PM +0200, Aurelien Jacobs wrote:
>> > > Hi,
>> > >
>> > > I want to apply the attached patch which provide support for Matroska
>> > > subtitle tracks.
>> > > Before applying it I wanted to be sure if it is OK to add
>> > > CODEC_ID_TEXT_SUBTITLE ?
>> > what is TEXT ?
>> > is it raw ASCII? raw UTF8? raw plain text (with some encoding specified
>> > somehow)?
>> I intended it to be raw UTF8, but that indeed needs to be clarified.
>> CODEC_ID_TEXT_UTF8_SUBTITLE seems clear enough to me. OK ?
> id call it CODEC_ID_PLAIN_TEXT_SUBTITLE with a comment saying that it is UTF8
> currently but we might add a text_encoding field in the future specifying
> the format
I'd call it simply CODEC_ID_TEXT. That it usually happens to be subtitle
text is completely irrelevant.
mans at mansr.com
More information about the ffmpeg-devel