[FFmpeg-devel] [PATCH] ffplay: remove remains of video_clock

[Marton Balint]

On Fri, 15 Feb 2013, Michael Niedermayer wrote:

> On Fri, Feb 15, 2013 at 12:51:13AM +0100, Marton Balint wrote:
>>
>> On Mon, 11 Feb 2013, Marton Balint wrote:
>>
>>> We are now using a pts based approach, libavutil/libavcodec should provide the
>>> correct pts-es anyway. This also fixes an issue when seeking to a frame with a
>>> pts set to zero.
>>>
>>> Signed-off-by: Marton Balint <cus at passwd.hu>
>>> ---
>>> ffplay.c |   29 ++++++-----------------------
>>> 1 file changed, 6 insertions(+), 23 deletions(-)
>>
>> Michael, please merge from ffplay stable for this patch and for the
>> patch I sent earlier to the list which improves the frame display
>> timing.
>>
>> b853cfe ffplay: remove refresh thread to achieve more precise frame delay
>> 5534d8f ffplay: remove remains of video_clock
>
> this seems to break seeking quite badly
> tested with
>
> ffplay matrixbench_mpeg2.mpg
> and seeking by clicking with the mouse and holding the button takes
> several seconds before seeking actually happens

I've just sent a new patch on top of the earlier two and also pushed it to 
github which should fix this. If it fixes the issue for you as well and 
nobody comments till the end of the day, then please merge the patch 
series.

Thanks,
Marton