[FFmpeg-devel] [PATCH] libavformat/matroskadec.c: Eliminate spurious "File is broken, keyframes not correctly marked!" in valid WebM

Michael Niedermayer michaelni at gmx.at
Fri Sep 20 03:49:32 CEST 2013


On Wed, Sep 11, 2013 at 03:37:18AM -0400, Monty Montgomery wrote:
> > I think it was supposed to warn for both cases.
> 
> Ah, OK.
> 
> > To my knowledge there were also files where nothing at all was marked as keyframe, causing seeking to not work at all.
> 
> I don't think that's an issue here.
> 
> > Is it really ok/sensible to not have a keyframe marked as such?
> 
> That doesn't appear to be the problem; the warning is tripped by audio
> blocks, not video.  That is,  skip_to_keyframe is set, and we land on
> an audio block... which, strictly speaking, is indeed supposed to
> flagged as a keyframe.

patch applied

thanks

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Everything should be made as simple as possible, but not simpler.
-- Albert Einstein
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20130920/54ec2dbc/attachment.asc>


More information about the ffmpeg-devel mailing list