Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

libVLC: Switch to using audio time stretching by default #1570

Merged

Conversation

DavidFair
Copy link
Contributor

@DavidFair DavidFair commented Mar 27, 2022

Changes
This matches libVLC's normal default. Looking back in the project
history this was set to no stretching from the very start @commit
9528b9a

Since libVLC uses it by default and this wasn't a bug fix I propose we
can opt in the audio stretching by default. This should only trigger when
a user selects a playback speed that isn't 1x anyway.

For future reference, the original changeset was here:
9528b9a#diff-226bb739f67bb7dbde6485e2433c55b1cc25fff3fea24029e5344b0c396657b6R232

Issues
None on GitHub, but I have seen it mentioned on Reddit and I was planning on doing this anyway so...

Testing

  • Ensure libVLC is forced
  • Change the speed to 2x
  • It should sound normal. Some cracks and pops are expected when the content and audio rate start to become CPU bound (such as using software emulation)

This matches libVLC's normal default. Looking back in the project
history this was set to no stretching from the very start @commit
9528b9a

Since libVLC use it by default and this wasn't a bug fix I propose we
can opt in the audio stretching by default. This should only trigger when
a user selects a playback speed that isn't 1x anyway.
@DavidFair DavidFair changed the title Switch to using audio time stretching by default libVLC: Switch to using audio time stretching by default Mar 27, 2022
@nielsvanvelzen nielsvanvelzen added this to In progress in v0.13.3 via automation Mar 27, 2022
v0.13.3 automation moved this from In progress to Reviewer approved Mar 27, 2022
@nielsvanvelzen nielsvanvelzen merged commit cfbadba into jellyfin:master Mar 27, 2022
v0.13.3 automation moved this from Reviewer approved to Done Mar 27, 2022
@DavidFair DavidFair deleted the VLC_Use_audio_time_stretch branch March 27, 2022 17:11
@nielsvanvelzen nielsvanvelzen moved this from Done to Cherrypicked in v0.13.3 Apr 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
v0.13.3
Cherrypicked
Development

Successfully merging this pull request may close these issues.

None yet

3 participants