From 1643cb865f4005949aa8a10b5eee7e0320f5ff4a Mon Sep 17 00:00:00 2001 From: wm4 Date: Thu, 3 Sep 2020 22:39:23 +0200 Subject: audio: fix stream-silence with push AOs (somewhat) --audio-stream-silence is a shitty feature compensating for awful consumer garbage, that mutes PCM at first to check whether it's compressed audio, using formats advocated and owned by malicious patent troll companies (who spend more money on their lawyers than paying any technicians), wrapped in a wasteful way to make it constant bitrate using a standard whose text is not freely available, and only rude users want it. This feature has been carelessly broken, because it's complicated and stupid. What would Jesus do? If not getting an aneurysm, or pushing over tables with expensive A/V receivers on top of them, he'd probably fix the feature. So let's take inspiration from Jesus Christ himself, and do something as dumb as wasting some of our limited lifetime on this incredibly stupid fucking shit. This is tricky, because state changes like end-of-audio are supposed to be driven by the AO driver, while playing silence precludes this. But it seems code paths for "untimed" AOs can be reused. But there are still problems. For example, underruns will just happen normally (and stop audio streaming), because we don't have a separate heuristic to check whether the buffer is "low enough" (as a consequence of a network stall, but before the audio output itself underruns). --- DOCS/man/options.rst | 5 +++++ 1 file changed, 5 insertions(+) (limited to 'DOCS') diff --git a/DOCS/man/options.rst b/DOCS/man/options.rst index 1107f3ecdd..3a092651ee 100644 --- a/DOCS/man/options.rst +++ b/DOCS/man/options.rst @@ -2116,6 +2116,11 @@ Audio Not all AOs support this. + .. admonition:: Warning + + This modifies certain subtle player behavior, like A/V-sync and underrun + handling. Enabling this option is strongly discouraged. + ``--audio-wait-open=`` This makes sense for use with ``--audio-stream-silence=yes``. If this option is given, the player will wait for the given amount of seconds after opening -- cgit v1.2.3