[FFmpeg-devel] [PATCH] matroska subtitle tracks support
Aurelien Jacobs
aurel
Wed Jul 11 15:14:05 CEST 2007
On Tue, 10 Jul 2007 17:51:20 +0100 (BST)
M?ns Rullg?rd <mans at mansr.com> wrote:
>
> Michael Niedermayer wrote:
> > Hi
> >
> > On Tue, Jul 10, 2007 at 01:29:11PM +0200, Aurelien Jacobs wrote:
> >> On Tue, 10 Jul 2007 11:01:48 +0100 (BST)
> >> M?ns Rullg?rd <mans at mansr.com> wrote:
> >>
> >> >
> >> > Aurelien Jacobs wrote:
> >> > > Indeed, I agree with this. So maybe CODEC_ID_PLAIN_TEXT ?
> >> >
> >> > "PLAIN" differentiating it from what? If we wanted an ID for, say, HTML
> >> > I'd suggest CODEC_ID_HTML etc. When no ambiguity exists, shorter names
> >> > should be preferred. This is not Java.
> >>
> >> That's fine with me. If Michael is ok, I will use CODEC_ID_TEXT.
> >
> > well, iam not opposed to CODEC_ID_TEXT though i still feel that
> > SUBTITLE should be part of the name but i am fine with CODEC_ID_TEXT
> > too if you prefer
>
> Why? We don't have CODEC_ID_H264_VIDEO or CODEC_ID_AC3_AUDIO. I fail
> to see what makes subtitles special. We have to use CODEC_ID_DVB_SUBTITLE
> since that encoding has no other non-ambiguous name.
Finally applied with CODEC_ID_TEXT.
Aurel
More information about the ffmpeg-devel
mailing list