From dc2d0539c78dea5d4a21f6bf79846c1bbedbb055 Mon Sep 17 00:00:00 2001 From: wm4 Date: Wed, 7 Jan 2015 22:23:38 +0100 Subject: ao_pulse: disable latency calculation hacks by default This used to be required to workaround PulseAudio bugs. Even later, when the bugs were (partially?) fixed in PulseAudio, I had the feeling the hacks gave better behavior. On the other hand, I couldn't actually reproduce any bad behavior without the hacks lately. On top of this, it seems our hacks sometimes perform much worse than PulseAudio's native implementation (see #1430). So disable the hacks by default, but still leave the code and the option in case it still helps somewhere. Also, being able to blame PulseAudio's code by using its native API is much easier than trying to debug our own (mplayer2-derived) hacks. --- DOCS/man/ao.rst | 9 +++++++-- audio/out/ao_pulse.c | 1 - 2 files changed, 7 insertions(+), 3 deletions(-) diff --git a/DOCS/man/ao.rst b/DOCS/man/ao.rst index ae91fd58ad..c78d812002 100644 --- a/DOCS/man/ao.rst +++ b/DOCS/man/ao.rst @@ -179,10 +179,15 @@ Available audio output drivers are: changes. Default: 250. ``latency-hacks=`` - Enable hacks to workaround PulseAudio timing bugs (default: yes). If + Enable hacks to workaround PulseAudio timing bugs (default: no). If enabled, mpv will do elaborate latency calculations on its own. If disabled, it will use PulseAudio automatically updated timing - information. Disabling this might help with e.g. networked audio. + information. Disabling this might help with e.g. networked audio or + some plugins, while enabling it might help in some unknown situations + (it used to be required to get good behavior on old PulseAudio versions). + + If you have stuttering video when using pulse, try to enable this + option. (Or alternatively, try to update PulseAudio.) ``dsound`` (Windows only) DirectX DirectSound audio output driver diff --git a/audio/out/ao_pulse.c b/audio/out/ao_pulse.c index ae708df59e..1f07189779 100644 --- a/audio/out/ao_pulse.c +++ b/audio/out/ao_pulse.c @@ -782,7 +782,6 @@ const struct ao_driver audio_out_pulse = { .priv_size = sizeof(struct priv), .priv_defaults = &(const struct priv) { .cfg_buffer = 250, - .cfg_latency_hacks = 1, }, .options = (const struct m_option[]) { OPT_STRING("host", cfg_host, 0), -- cgit v1.2.3