From ee4b14322155b5808eeceb463f5edcd751eb3a98 Mon Sep 17 00:00:00 2001 From: Marton Balint Date: Thu, 15 Mar 2012 00:41:57 +0100 Subject: [PATCH] ffplay: use frame count based queueing for audio queue This reduces the number of queued frames for audio data but also reduces the amount of A-V difference after changing the audio stream (because less frames are queued). Fixes bug #1035. Signed-off-by: Marton Balint --- ffplay.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/ffplay.c b/ffplay.c index 8a5818f466..00be13de43 100644 --- a/ffplay.c +++ b/ffplay.c @@ -64,7 +64,6 @@ const char program_name[] = "ffplay"; const int program_birth_year = 2003; #define MAX_QUEUE_SIZE (15 * 1024 * 1024) -#define MIN_AUDIOQ_SIZE (20 * 16 * 1024) #define MIN_FRAMES 5 /* SDL audio buffer size, in samples. Should be small to have precise @@ -2612,7 +2611,7 @@ static int read_thread(void *arg) /* if the queue are full, no need to read more */ if ( is->audioq.size + is->videoq.size + is->subtitleq.size > MAX_QUEUE_SIZE - || ( (is->audioq .size > MIN_AUDIOQ_SIZE || is->audio_stream < 0) + || ( (is->audioq .nb_packets > MIN_FRAMES || is->audio_stream < 0) && (is->videoq .nb_packets > MIN_FRAMES || is->video_stream < 0) && (is->subtitleq.nb_packets > MIN_FRAMES || is->subtitle_stream < 0))) { /* wait 10 ms */