Re: [FFmpeg-devel] [PATCH] avformat/oggdec: only parse headers before data

2019-06-19 Thread James Almer
On 6/19/2019 11:11 PM, Chris Cunningham wrote: > On Wed, Jun 19, 2019 at 11:25 AM Michael Niedermayer > wrote: > > breaks: > ./ffmpeg -i bgc.sub.dub.ogm -vframes 3 -y test.webm > sample: http://samples.mplayerhq.hu/ogg/bgc.sub.dub.ogm > > [...] > > -- > Michael     GnuP

Re: [FFmpeg-devel] [PATCH] avformat/oggdec: only parse headers before data

2019-06-19 Thread Chris Cunningham
On Wed, Jun 19, 2019 at 7:11 PM Chris Cunningham wrote: > On Wed, Jun 19, 2019 at 11:25 AM Michael Niedermayer > wrote: > >> breaks: >> ./ffmpeg -i bgc.sub.dub.ogm -vframes 3 -y test.webm >> sample: http://samples.mplayerhq.hu/ogg/bgc.sub.dub.ogm >> >> [...] >> >> -- >> Michael GnuPG fingerp

Re: [FFmpeg-devel] [PATCH] avformat/oggdec: only parse headers before data

2019-06-19 Thread Chris Cunningham
On Wed, Jun 19, 2019 at 11:25 AM Michael Niedermayer wrote: > breaks: > ./ffmpeg -i bgc.sub.dub.ogm -vframes 3 -y test.webm > sample: http://samples.mplayerhq.hu/ogg/bgc.sub.dub.ogm > > [...] > > -- > Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB > Thanks Michael. That

Re: [FFmpeg-devel] [PATCH] avformat/oggdec: only parse headers before data

2019-06-19 Thread Michael Niedermayer
On Mon, Jun 17, 2019 at 05:59:40PM -0700, Chris Cunningham wrote: > This behavior was added in 2010 to suport some old (and invalid) ogm > files. > https://github.com/FFmpeg/FFmpeg/commit/81b743eb1026547270b88ac6a5cb451a3907ee94 > > But this makes it possible to change the codec in the later head

Re: [FFmpeg-devel] [PATCH] avformat/oggdec: only parse headers before data

2019-06-17 Thread Chris Cunningham
+James On Mon, Jun 17, 2019 at 6:31 PM Chris Cunningham wrote: > This behavior was added in 2010 to suport some old (and invalid) ogm > files. > https://github.com/FFmpeg/FFmpeg/commit/81b743eb1026547270b88ac6a5cb451a3907ee94 > > But this makes it possible to change the codec in the later header

[FFmpeg-devel] [PATCH] avformat/oggdec: only parse headers before data

2019-06-17 Thread Chris Cunningham
This behavior was added in 2010 to suport some old (and invalid) ogm files. https://github.com/FFmpeg/FFmpeg/commit/81b743eb1026547270b88ac6a5cb451a3907ee94 But this makes it possible to change the codec in the later headers, causing codec to be out of sync with internal avctx (eventually trigger