Skip to content

sub: add sub-lines property#17730

Open
guidocella wants to merge 2 commits intompv-player:masterfrom
guidocella:sub-lines
Open

sub: add sub-lines property#17730
guidocella wants to merge 2 commits intompv-player:masterfrom
guidocella:sub-lines

Conversation

@guidocella
Copy link
Copy Markdown
Contributor

@guidocella guidocella commented Apr 9, 2026

sub: add sub-lines property

Return all subtitle lines in memory with text, start and end.

I did not name this sub-all so it cannot be misinterpreted as referring to all subtitle tracks.

Closes #12810.

Mostly generated by Claude.

select.lua: use sub-lines instead of forking to ffmpeg

This:

  • Removes the dependency of ffmpeg in PATH - useful on Windows, macOS (?), Flatpak.
  • Can support subrandr (not yet implemented).
  • Avoids redownloading network subtitles.
  • Avoids slow demuxing of large mkvs.
  • But no longer returns future lines of embedded subtitles.
  • No longer returns embedded lines of regions you seeked beyond without playing them (but holding the up key works) this depends if you enable the cache.
  • Changing sub track drops cached subtitles.

Return all subtitle lines in memory with text, start and end.

I did not name this sub-all so it cannot be misinterpreted as referring
to all subtitle tracks.

Closes mpv-player#12810.
This:

- Removes the dependency of ffmpeg in PATH - useful on Windows, macOS
  (?), Flatpak.
- Can support subrandr (not yet implemented).
- Avoids redownloading network subtitles.
- Avoids slow demuxing of large mkvs.
- But no longer returns future lines of embedded subtitles.
- No longer returns embedded lines of regions you seeked beyond without
  playing them.
- Changing sub track drops cached subtitles.
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.

Add sub-text-all property

1 participant