Re: [FFmpeg-devel] [PATCH 1/3] ffplay: update frame timer based on last updated clock time when toggling pause

2015-02-10 Thread Marton Balint


On Fri, 6 Feb 2015, Marton Balint wrote:


It is better than using simply video clock, because video clock may be NAN.

Signed-off-by: Marton Balint c...@passwd.hu
---


Hello Michael,

Please merge from my stable branch for the ffplay patch series:

6885009 ffplay: update frame timer based on last updated clock time when 
toggling pause
d6910c4 ffplay: make eof part of videostate and signal it when opening a stream
ef1ccea ffplay: factorize thread starting and stopping code into decoder

Thanks,
Marton


___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel


Re: [FFmpeg-devel] [PATCH 1/3] ffplay: update frame timer based on last updated clock time when toggling pause

2015-02-10 Thread Michael Niedermayer
On Tue, Feb 10, 2015 at 09:52:18PM +0100, Marton Balint wrote:
 
 On Fri, 6 Feb 2015, Marton Balint wrote:
 
 It is better than using simply video clock, because video clock may be NAN.
 
 Signed-off-by: Marton Balint c...@passwd.hu
 ---
 
 Hello Michael,
 
 Please merge from my stable branch for the ffplay patch series:
 
 6885009 ffplay: update frame timer based on last updated clock time when 
 toggling pause
 d6910c4 ffplay: make eof part of videostate and signal it when opening a 
 stream
 ef1ccea ffplay: factorize thread starting and stopping code into decoder

merged

thanks

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

Rewriting code that is poorly written but fully understood is good.
Rewriting code that one doesnt understand is a sign that one is less smart
then the original author, trying to rewrite it will not make it better.


signature.asc
Description: Digital signature
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel


[FFmpeg-devel] [PATCH 1/3] ffplay: update frame timer based on last updated clock time when toggling pause

2015-02-05 Thread Marton Balint
It is better than using simply video clock, because video clock may be NAN.

Signed-off-by: Marton Balint c...@passwd.hu
---
 ffplay.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/ffplay.c b/ffplay.c
index c112ead..210bae5 100644
--- a/ffplay.c
+++ b/ffplay.c
@@ -1494,7 +1494,7 @@ static void stream_seek(VideoState *is, int64_t pos, 
int64_t rel, int seek_by_by
 static void stream_toggle_pause(VideoState *is)
 {
 if (is-paused) {
-is-frame_timer += av_gettime_relative() / 100.0 + 
is-vidclk.pts_drift - is-vidclk.pts;
+is-frame_timer += av_gettime_relative() / 100.0 - 
is-vidclk.last_updated;
 if (is-read_pause_return != AVERROR(ENOSYS)) {
 is-vidclk.paused = 0;
 }
-- 
2.1.4

___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel