Re: [FFmpeg-devel] [PATCH] avformat/mov: add VP8 codec support

2018-02-06 Thread James Almer
On 2/6/2018 9:25 PM, Carl Eugen Hoyos wrote: > 2018-02-06 2:39 GMT+01:00 James Almer : >> Demuxing only. Muxing is disabled as altref frame handling is not >> defined, and there's no way to know the presence of such frames >> during stream initialization. > > strict -2? Allowing muxing something

Re: [FFmpeg-devel] [PATCH] avformat/mov: add VP8 codec support

2018-02-06 Thread Carl Eugen Hoyos
2018-02-06 2:39 GMT+01:00 James Almer : > Demuxing only. Muxing is disabled as altref frame handling is not > defined, and there's no way to know the presence of such frames > during stream initialization. strict -2? > +if (track->par->codec_id == AV_CODEC_ID_VP8) { > +

[FFmpeg-devel] [PATCH] avformat/mov: add VP8 codec support

2018-02-05 Thread James Almer
Demuxing only. Muxing is disabled as altref frame handling is not defined, and there's no way to know the presence of such frames during stream initialization. Based on a patch by Steven Liu. Fixes ticket #7000 Signed-off-by: James Almer --- libavformat/isom.c | 1 + libavformat/mov.c| 1

Re: [FFmpeg-devel] [PATCH] avformat/mov: add VP8 codec support

2018-02-05 Thread Carl Eugen Hoyos
2018-02-05 6:52 GMT+01:00 Steven Liu : > fix ticket: 7000 I believe this patch would allow to create invalid mp4 files containing vp8, so it cannot get committed. See 54327c2b Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpe

[FFmpeg-devel] [PATCH] avformat/mov: add VP8 codec support

2018-02-04 Thread Steven Liu
fix ticket: 7000 Signed-off-by: Steven Liu --- libavformat/isom.c | 1 + libavformat/mov.c | 1 + 2 files changed, 2 insertions(+) diff --git a/libavformat/isom.c b/libavformat/isom.c index 9d9f85885b..95dceaa50b 100644 --- a/libavformat/isom.c +++ b/libavformat/isom.c @@ -187,6 +187,7 @@ cons