At 2016-09-06T22:18:18-0400, Ronald S. Bultje wrote:
> I think the patch is fine, but I wonder if it should set it to _FULL (the
> parser ignores the option, but it is semantically more correct).

Good point, thanks for the feedback.  Updated (simpler!) patch attached.

Cheers,

>From a9d69f145bb92939c1ffce11a0522dffff76857f Mon Sep 17 00:00:00 2001
From: Matthew Gregan <kine...@flim.org>
Date: Wed, 7 Sep 2016 13:58:30 +1200
Subject: avformat/mov: Enable stream parsing for VP9.

MP4 media containing VP9 using superframes (such as
https://github.com/Netflix/vp9-dash/raw/master/DASH-Samples/Fountain_2997_0560kbps_640x480_4x3PAR.ivf_DashUnencrypted.ismv)
does not decode correctly with the built-in VP9 decoder because
superframes are passed to the decoder whole rather than split into
individual frames.

Signed-off-by: Matthew Gregan <kine...@flim.org>

diff --git a/libavformat/mov.c b/libavformat/mov.c
index f499906..fa4e016 100644
--- a/libavformat/mov.c
+++ b/libavformat/mov.c
@@ -2161,6 +2161,7 @@ static int mov_finalize_stsd_codec(MOVContext *c, AVIOContext *pb,
     case AV_CODEC_ID_EAC3:
     case AV_CODEC_ID_MPEG1VIDEO:
     case AV_CODEC_ID_VC1:
+    case AV_CODEC_ID_VP9:
         st->need_parsing = AVSTREAM_PARSE_FULL;
         break;
     default:
-- 
2.9.3

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to