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

[Merge pull request] Add support for SSA MarginL, MarginR, MarginV #1482

Closed
sand-bit opened this issue Jun 22, 2024 · 1 comment
Closed

[Merge pull request] Add support for SSA MarginL, MarginR, MarginV #1482

sand-bit opened this issue Jun 22, 2024 · 1 comment
Assignees

Comments

@sand-bit
Copy link

Use case description

Better SSA/ASS subtitle formatting support. This is an area where ExoPlayer/Media3 has lagged behind VLC.
There was this issue tracker for SSA format support: google/ExoPlayer#8435 (comment)
VLC achieves SSA rendering via https://github.com/libass/libass however because it was written in C, there was reluctance to use it.

A pull request has been sitting in the ExoPlayer report without being merged: google/ExoPlayer#10169

Proposed solution

Merge google/ExoPlayer#10169 and further improve SSA styling support using libass as reference and complete the support matrix here: google/ExoPlayer#8435 (comment)

Alternatives considered

Use libass like VLC

@icbaker
Copy link
Collaborator

icbaker commented Jun 25, 2024

We can't merge google/ExoPlayer#10169 in its current form because the ExoPlayer GitHub project is deprecated and no longer updated.

When I tried merging it back in 2022 I'm afraid it became too complex and I didn't have the capacity to resolve all the open questions.

If someone would like to re-open a PR in this project with the open questions on google/ExoPlayer#10169 resolved then we can take another look.

I'm going to close this issue as a duplicate of google/ExoPlayer#8435 since it has more info.

I'm afraid we continue to not consider using native code for handling user-generated content except where performance concerns preclude this. This adds technical complexity to our build process, and also introduces security risks due to lack of memory safety in C(++).

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

No branches or pull requests

2 participants