[MPlayer-dev-eng] mencoder frame duplication bug

[Diego Biurrun]

Hi Jan!

Please don't forget to put [PATCH] in the subject, patches have a
tendency to get overlooked and forgotten if you don't.  It's all
explained in DOCS/tech/patches.txt.
Thanks

Diego

P.S.: Will you send in the demuxer for that exotic mbone file format
you wrote?  Don't remember its name..

Jan Kritzner writes:
 > I've used mencoder to transcode videos with variable frame-rates to avi-Files 
 > with a fixed frame-rate.
 > The input file was a video with a frame rate of 10fps and presentation 
 > timestamps in the file. Assume that frames 10-19 have been lost.
 > 
 > MPlayer does the right thing, presents frame 9 for 1.1 seconds, and continues 
 > with frame 20.
 > 
 > MEncoder calculates the right "frame_time", but duplicates the frame following 
 > the gap, i.e. frame 20 is presented for 1.1 seconds.
 > 
 > FIX:
 > 
 > Update
 > "v_timer_corr-=frame_time-(float)mux_v->h.dwScale/mux_v->h.dwRate;"
 > before the number of skips is calculated.
 > 
 > Should be a trivial fix, but I haven't looked at side-effects.
 > Jan