On Thu, 28 Jun 2012 23:11:31 -0700, Alex Converse <[email protected]> 
wrote:
> On Tue, Jun 26, 2012 at 10:07 AM, Anton Khirnov <[email protected]> wrote:
> > ---
> >  libavformat/mov.c |   49 +++++++++++++++++++++++++++++++++++++++++++++++++
> >  1 file changed, 49 insertions(+)
> >
> 
> Is this documented anywhere. I don't see it in the QTFF spec.
> 

Didn't find any docs anywhere.
This patch is based on a few samples i was able to obtain and googled
information.

> > diff --git a/libavformat/mov.c b/libavformat/mov.c
> > index 6325401..8434da0 100644
> > --- a/libavformat/mov.c
> > +++ b/libavformat/mov.c
> > @@ -166,6 +166,47 @@ static int mov_read_mac_string(MOVContext *c, 
> > AVIOContext *pb, int len,
> >     return p - dst;
> >  }
> >
> > +static int mov_read_covr(MOVContext *c, AVIOContext *pb, int type, int len)
> > +{
> > +    AVPacket pkt;
> > +    AVStream *st;
> > +    MOVStreamContext *sc;
> > +    enum CodecID id;
> > +    int ret;
> > +
> > +    switch (type) {
> > +    case 0xd: id = CODEC_ID_MJPEG; break;
> > +    case 0xe: id = CODEC_ID_PNG;   break;
> > +    default:
> > +        av_log(c->fc, AV_LOG_WARNING, "Unknown cover type: 0x%x.\n", type);
> > +        avio_skip(pb, len);
> > +        return 0;
> > +    }
> > +
> 
> slightly surprised that there is no TIFF option
> 

The samples i found are all jpg, but some google results also mention
png, but nothing else.

In any case, we can add support for tiff later if we find samples.

-- 
Anton Khirnov
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to