Skip to content

player/loadfile: possibly do a reset of sd_ass when turning on video#17278

Merged
Dudemanguy merged 2 commits intompv-player:masterfrom
Dudemanguy:animated-packet-fix
Jan 21, 2026
Merged

player/loadfile: possibly do a reset of sd_ass when turning on video#17278
Dudemanguy merged 2 commits intompv-player:masterfrom
Dudemanguy:animated-packet-fix

Conversation

@Dudemanguy
Copy link
Member

No description provided.

This should be done otherwise it could technically increment forever if
you constant reset.
It's a pretty niche case, but if you start a file with no video with
animated ass subtitles and then turned on the video later, the subtitles
would not actually be rendered animated. This because the subtitles were
already decoded, and we do not do a check for animations expect in the
specific case of an image being played back on the video since the check
can be quite expensive. To fix this edge case, we just need to reset the
ass before doing the track switch in the case where we go from no video
to a track that's an image. SD_CTRL_RESET_SOFT is added as a a sub_ctrl
to accomplish this. Fixes mpv-player#17268.
Copy link
Member

@kasper93 kasper93 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGMT, thanks!

@Dudemanguy Dudemanguy merged commit 49b6d80 into mpv-player:master Jan 21, 2026
30 checks passed
@Dudemanguy Dudemanguy deleted the animated-packet-fix branch January 21, 2026 15:18
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.

2 participants