[vlc-commits] pulse: let pulseaudio define prebuffer/ minrequest and use that as buffersize, also change maxlength to 100ms

[Ilkka Ollakka]

vlc | branch: master | Ilkka Ollakka <ileoo at videolan.org> | Sat Mar 26 13:44:14 2011 +0200| [100fb7292499805025523060e37f006acb153d22] | committer: Ilkka Ollakka

pulse: let pulseaudio define prebuffer/minrequest and use that as buffersize, also change maxlength to 100ms

It's more along pulseaudio recommendations, and we don't really need to define those
ourself. Maxlength is defined to keep latency in someway in limits. Please tell if
this acts for you.

> http://git.videolan.org/gitweb.cgi/vlc.git/?a=commit;h=100fb7292499805025523060e37f006acb153d22
---

 modules/audio_output/pulse.c |   22 +++++++++++-----------
 1 files changed, 11 insertions(+), 11 deletions(-)

diff --git a/modules/audio_output/pulse.c b/modules/audio_output/pulse.c
index e634c4e..710bab2 100644
--- a/modules/audio_output/pulse.c
+++ b/modules/audio_output/pulse.c
@@ -182,17 +182,14 @@ static int Open ( vlc_object_t *p_this )
         goto fail;
     }
 
-    /* Reduce overall latency to 200mS to reduce audible clicks
+    /* Reduce overall latency to 100mS to reduce audible clicks
      * Also pulse minreq and internal buffers are now 100mS which reduces resampling
      * but still shouldn't drop samples with some usb sound cards
      */
-    a.tlength = pa_bytes_per_second(&ss)/5;
+    a.tlength = pa_bytes_per_second(&ss)/10;
     a.maxlength = a.tlength * 2;
-    a.prebuf = a.tlength / 2;
-    a.minreq = a.tlength / 2;
-
-    /* Buffer size is 100mS */
-    p_sys->buffer_size = a.minreq;
+    a.prebuf = -1;
+    a.minreq = -1;
 
     /* Initialise the speaker map setup above */
     pa_channel_map_init_auto(&map, ss.channels, PA_CHANNEL_MAP_ALSA);
@@ -278,6 +275,9 @@ static int Open ( vlc_object_t *p_this )
 
     buffer_attr = pa_stream_get_buffer_attr(p_sys->stream);
     p_aout->output.i_nb_samples = buffer_attr->minreq / pa_frame_size(&ss);
+
+    /* Set buffersize from pulseaudio defined minrequest */
+    p_sys->buffer_size = buffer_attr->minreq;
     p_aout->output.pf_play = Play;
     aout_VolumeSoftInit(p_aout);
     msg_Dbg(p_aout, "Pulse initialized successfully");
@@ -476,10 +476,10 @@ static void stream_request_cb(pa_stream *s, size_t length, void *userdata) {
         else
         {
             PULSE_DEBUG( "Pulse stream request write zeroes");
-            void *data = pa_xmalloc(buffer_size);
-            memset(data, 0, buffer_size);
-            pa_stream_write(p_sys->stream, data, buffer_size, pa_xfree, 0, PA_SEEK_RELATIVE);
-            length -= buffer_size;
+            void *data = pa_xmalloc(length);
+            memset(data, 0, length);
+            pa_stream_write(p_sys->stream, data, length, pa_xfree, 0, PA_SEEK_RELATIVE);
+            length = 0;
         }
     }while(length > buffer_size);