Skip to content

Conversation

@fritsch
Copy link
Member

@fritsch fritsch commented Feb 17, 2024

Backport of #24597

Kodi can supervise the sink for being stuck, which means: if you open a sink with 160 ms buffer and it does not move while eating more than twice of the buffer's audio data, it is considered a bug. Sadly - there is a whole lot of broken firmwares out there, one of the leards: Ugoos. Which is the reason I make disabled by default. That means that FireTV Cube 3rd Gen and others would need to enable this - once again - via an advanced settings.

<advancedsettings>
<audio>
<superviseaudiodelay>true</superviseaudiodelay>
<audio>
</advancedsettings>

This is a backport towards Nexus as thie "good meant" supervising by default caused regresions, on you guessed it, broken firmware or semi-broken firmware (if you consider 300 ms off-sync semi-broken ;-)). Use the same "opt-in" as in master / v21.

@fritsch fritsch requested a review from thexai February 17, 2024 19:33
@fritsch fritsch added v20 Nexus Type: Fix non-breaking change which fixes an issue Platform: Android Component: Audio labels Feb 17, 2024
@fritsch fritsch added this to the Nexus 20.5 milestone Feb 17, 2024
Kodi can supervise the sink for being stuck, which means: if you open a sink
with 160 ms buffer and it does not move while eating more than twice of the
buffer's audio data, it is considered a bug. Sadly - there is a whole
lot of broken firmwares out there, one of the leards: Ugoos. Which is the
reason I make disabled by default. That means that FireTV Cube 3rd Gen and
others would need to enable this - once again - via an advanced settings.

<advancedsettings>
<audio>
<superviseaudiodelay>true</superviseaudiodelay>
<audio>
</advancedsettings>

Co-authored-by: thexai <58434170+thexai@users.noreply.github.com>
@fritsch
Copy link
Member Author

fritsch commented Feb 17, 2024

The cherry-pick forgot some includes, which were coming from another change before it - fixed it.

@fritsch fritsch merged commit 2de1546 into xbmc:Nexus Feb 18, 2024
@rainman74
Copy link

rainman74 commented Mar 2, 2024

<superviseaudiodelay>true</superviseaudiodelay>
is automatically changed by Kodi to
<superviseaudiodelay></superviseaudiodelay>

Why is that?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants