Hi Hendrik, On Nov 2, 2017 4:21 AM, "Hendrik Leppkes" <[email protected]> wrote:
On Thu, Nov 2, 2017 at 3:10 AM, Sean McGovern <[email protected]> wrote: > --- > libavformat/matroskadec.c | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/libavformat/matroskadec.c b/libavformat/matroskadec.c > index c6e1a19..3743d4d 100644 > --- a/libavformat/matroskadec.c > +++ b/libavformat/matroskadec.c > @@ -1606,6 +1606,10 @@ static int mkv_parse_video_projection(AVStream *st, const MatroskaTrack *track) > int ret; > GetByteContext gb; > > + if (track->video.stereo_mode == MATROSKA_VIDEO_STEREOMODE_TYPE_NB) { > + return 0; > + } > + Stereo-mode seems a bit unrelated to the projection, additionally comparing to the _NB constant seems wrong, it should never be that value. The sample file in question predates both stereo mode and spherical video streams and yet currently it triggers the log warning: Unknown spherical metadata type 0 from the default: clause of the switch statement further down in this function. Thinking about this it probably shouldn't even be calling this function at all in this situation. Will look at it again and propose a new patch. Please consider this one dropped for now. _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel -- Sean McG. _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
