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

[Michael Niedermayer]

On Fri, Feb 15, 2013 at 01:40:15PM +0100, Marton Balint wrote:
> 
> 
> 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.

tested, works, end of day reached
merged

thanks

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

The worst form of inequality is to try to make unequal things equal.
-- Aristotle
-------------- 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/20130216/e2be8eb5/attachment.asc>