[FFmpeg-devel] [PATCH] mpegvideo_parser: Fix SEQ_END handling.

Michael Niedermayer michaelni at gmx.at
Sun Jan 22 15:42:12 CET 2012


On Sun, Jan 22, 2012 at 03:31:11PM +0100, Reimar Döffinger wrote:
> On Sun, Jan 22, 2012 at 03:23:07PM +0100, Michael Niedermayer wrote:
> > This restores the seq end handling so that seq ends are considered
> > seperate frames and cause the last frame to be flushed.
> 
> sepArate.

fixed


> So you found a sample? Sorry, I forgot to look for one.

just what hendrik posted


> If it works better I guess it is ok, however it wouldn't hurt
> if we had comments when things depend on a non-obvious way on
> the order of code as _seems_ to be the case here.

> Adding a sample to fate wouldn't hurt either.

iam not sure how to do that, the difference is when the frame is
output not what or in what order it is output

previously the frame would be output when the next frame or end is
reached
after the patch it would be output when the SEQ_END is found

If this fixes some real case i do not know ... i need some feedback
here but the way its after the patch is how the issue was fixed
originally if i dont misremember

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

You can kill me, but you cannot change the truth.
-------------- 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/20120122/d5711800/attachment.asc>


More information about the ffmpeg-devel mailing list