[FFmpeg-devel] [PATCH]Basic XSUB encoder (take 5)

Michael Niedermayer michaelni
Fri Jun 19 13:50:16 CEST 2009


On Fri, Jun 19, 2009 at 12:21:55PM +0200, Reimar D?ffinger wrote:
> On Thu, Jun 18, 2009 at 03:56:49PM +0200, Michael Niedermayer wrote:
> > now the avi changes ...
> > 
> > [...]
> > > Index: libavformat/avienc.c
> > > ===================================================================
> > > --- libavformat/avienc.c	(revision 18911)
> > > +++ libavformat/avienc.c	(working copy)
> > > @@ -81,6 +81,9 @@
> > >      if (type == CODEC_TYPE_VIDEO) {
> > >          tag[2] = 'd';
> > >          tag[3] = 'c';
> > > +    } else if (type == CODEC_TYPE_SUBTITLE) {
> > > +        tag[2] = 's';
> > > +        tag[3] = 'b';
> > >      } else {
> > >          tag[2] = 'w';
> > >          tag[3] = 'b';
> > > @@ -212,8 +215,10 @@
> > >          case CODEC_TYPE_AUDIO: put_tag(pb, "auds"); break;
> > >  //        case CODEC_TYPE_TEXT : put_tag(pb, "txts"); break;
> > >          case CODEC_TYPE_DATA : put_tag(pb, "dats"); break;
> > > +        case CODEC_TYPE_SUBTITLE: put_tag(pb, "vids"); break;
> > 
> > can you quote some spec that requires these?
> > 
> > 
> > [...]
> > > @@ -253,6 +258,7 @@
> > >          strf = start_tag(pb, "strf");
> > >          switch(stream->codec_type) {
> > >          case CODEC_TYPE_VIDEO:
> > > +        case CODEC_TYPE_SUBTITLE:
> > >              put_bmp_header(pb, stream, codec_bmp_tags, 0);
> > >              break;
> > >          case CODEC_TYPE_AUDIO:
> > 
> > same question
> > 
> > if its just xsub then there should be a check for that not just the
> > codec_type IMHO
> 
> I don't have a device to actually test the result, but how does attached
> patch look to you?

ok

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I have often repented speaking, but never of holding my tongue.
-- Xenocrates
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090619/bab08a96/attachment.pgp>



More information about the ffmpeg-devel mailing list