[FFmpeg-devel] [PATCH]lavf/mov: Support alac extradata in wave atom v2

Michael Niedermayer michael at niedermayer.cc
Tue Aug 18 21:24:22 CEST 2015


On Mon, Aug 17, 2015 at 07:59:56PM +0000, Carl Eugen Hoyos wrote:
> Michael Niedermayer <michael <at> niedermayer.cc> writes:
> 
> > +            ffio_ensure_seekback(pb, 8);
> > +            buffer = avio_rb64(pb);
> > +            atom.size -= 8;
> > +            if (  (buffer & 0xFFFFFFFF) == MKBETAG('f','r','m','a')
> > +                && buffer >> 32 <= atom.size
> > +                && buffer >> 32 >= 8) {
> > +                avio_skip(pb,  - 8);
> > +                atom.size += 8;
> 
> I tested that QT expects the frma atom first 
> in the wave atom, so this variant is ok as 
> far as I can tell.

feel free to push the variant you tested

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

It is dangerous to be right in matters on which the established authorities
are wrong. -- Voltaire
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20150818/0a7a0eaf/attachment.sig>


More information about the ffmpeg-devel mailing list