[FFmpeg-devel] [PATCH] rmdec.c: merge old/new packet reading code

Kostya kostya.shishkov
Tue Mar 10 20:04:40 CET 2009

On Tue, Mar 10, 2009 at 02:29:11PM -0400, Ronald S. Bultje wrote:
> Hi,
> On Tue, Mar 10, 2009 at 1:58 PM, Kostya <kostya.shishkov at gmail.com> wrote:
> > Looks like some leftover from dead code, so you may start thinking
> > of getting read of it completely ;)
> As you wish. You'll see the bottom part of this patch shows the only
> useful use of seq, but that condition (in any version that I've ever
> seen) is always true, so not sure what the use is... I just removed it
> so we index only keyframes, I suppose that was the intention anyway.
> Ronald

patch looks ok

And from what I remember seq might have been used for marking as keyframe
only the video packets that contained first slice of video frame.

More information about the ffmpeg-devel mailing list