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

2015-08-26 Thread Carl Eugen Hoyos
Michael Niedermayer niedermayer.cc> writes: > feel free to push the variant you tested Done, thank you! Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

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

2015-08-18 Thread Michael Niedermayer
On Mon, Aug 17, 2015 at 07:59:56PM +, Carl Eugen Hoyos wrote: > Michael Niedermayer niedermayer.cc> writes: > > > +ffio_ensure_seekback(pb, 8); > > +buffer = avio_rb64(pb); > > +atom.size -= 8; > > +if ( (buffer & 0x) == MKBETAG('f','r'

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

2015-08-17 Thread Carl Eugen Hoyos
Michael Niedermayer niedermayer.cc> writes: > +ffio_ensure_seekback(pb, 8); > +buffer = avio_rb64(pb); > +atom.size -= 8; > +if ( (buffer & 0x) == MKBETAG('f','r','m','a') > +&& buffer >> 32 <= atom.size > +&

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

2015-08-06 Thread Michael Niedermayer
On Wed, Aug 05, 2015 at 11:52:43PM +0200, Carl Eugen Hoyos wrote: > On Wednesday 05 August 2015 02:39:38 pm Michael Niedermayer wrote: > > On Mon, Aug 03, 2015 at 11:16:12PM +0200, Carl Eugen Hoyos wrote: > > > Hi! > > > > > > Attached patch fixes ticket #4747 and should not introduce any > > > the

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

2015-08-05 Thread Carl Eugen Hoyos
On Wednesday 05 August 2015 02:39:38 pm Michael Niedermayer wrote: > On Mon, Aug 03, 2015 at 11:16:12PM +0200, Carl Eugen Hoyos wrote: > > Hi! > > > > Attached patch fixes ticket #4747 and should not introduce any > > theoretical regressions. > > > > Please review, Carl Eugen > > > > mov.c | 22

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

2015-08-05 Thread Michael Niedermayer
On Mon, Aug 03, 2015 at 11:16:12PM +0200, Carl Eugen Hoyos wrote: > Hi! > > Attached patch fixes ticket #4747 and should not introduce any > theoretical regressions. > > Please review, Carl Eugen > mov.c | 22 ++ > 1 file changed, 22 insertions(+) > ffe1a030382f6bc11a5a58a

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

2015-08-03 Thread Carl Eugen Hoyos
Hi! Attached patch fixes ticket #4747 and should not introduce any theoretical regressions. Please review, Carl Eugendiff --git a/libavformat/mov.c b/libavformat/mov.c index 154d2f8..f3cb71f 100644 --- a/libavformat/mov.c +++ b/libavformat/mov.c @@ -1435,6 +1435,28 @@ static int mov_read_wave(MO