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

Extract external subs from container before determining characterset #8182

Merged
merged 2 commits into from Aug 12, 2022

Conversation

Shadowghost
Copy link
Contributor

@Shadowghost Shadowghost commented Jul 27, 2022

Subtitle character set determination requires the subtitle to not be in a container. Timestamps should also not be reset to start at zero when extracting subtitles.

Changes

  • Extract subtitle from external subtitle container (mks) before processing it
  • Do not reset Subtitle start time to zero when extracting

Fixes #8157

@Shadowghost Shadowghost added the stable backport Backport into the next stable release label Jul 27, 2022
@Bond-009 Bond-009 merged commit 1faee43 into jellyfin:release-10.8.z Aug 12, 2022
jellyfin-bot pushed a commit that referenced this pull request Aug 14, 2022
Extract external subs from container before determining characterset

Original-merge: 1faee43

Merged-by: Bond-009 <bond.009@outlook.com>

Backported-by: Joshua Boniface <joshua@boniface.me>
@jellyfin-bot jellyfin-bot removed the stable backport Backport into the next stable release label Aug 14, 2022
Kakadus pushed a commit to Kakadus/jellyfin that referenced this pull request Oct 2, 2022
Extract external subs from container before determining characterset

Original-merge: 1faee43

Merged-by: Bond-009 <bond.009@outlook.com>

Backported-by: Joshua Boniface <joshua@boniface.me>
@Shadowghost Shadowghost deleted the fix-sub-characterset branch October 14, 2022 07:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants