[MPlayer-dev-eng] [PATCH] Choose keyframe for edits in MOV demuxer

[John Koleszar]

Reimar Döffinger wrote:
> Hello,
>> The patch works fine on all samples I tried.  Any objections or comments
>> on it?
> 
> I don't exactly object, but IMHO seeking to the frame the EDL specifies
> is the correct behaviour and the person creating the EDL should make
> sure it points to a keyframe if that is needed (I think you could create
> a file where the destination does not need to be a keyframe).
> 

You could very well be correct - I don't know if the file is malformed or not.
If it is well formed, then the proper playback behavior is probably to seek to
the prior keyframe and then decode without displaying all frames prior to the
start of the edit. I don't have the code in front of me - can a demuxer signal
mplayer/mencoder that a packet should not be rendered? I'll look into how hard
it would be to implement this behavior.

Maybe somebody who knows the QT spec could say what the proper semantics are?

John