[PR #470] [MERGED] Added basic ASS/SSA encoding support #1287

Closed
opened 2026-01-29 17:00:52 +00:00 by claunia · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/CCExtractor/ccextractor/pull/470
Author: @Izaron
Created: 12/2/2016
Status: Merged
Merged: 12/2/2016
Merged by: @cfsmp3

Base: masterHead: ass-ssa-2


📝 Commits (1)

  • f8e3788 Added basic ASS/SSA encoding support

📊 Changes

8 files changed (+240 additions, -4 deletions)

View changed files

📝 src/lib_ccx/ccx_common_constants.h (+1 -0)
📝 src/lib_ccx/ccx_common_option.h (+1 -0)
📝 src/lib_ccx/ccx_decoders_common.c (+6 -3)
📝 src/lib_ccx/ccx_encoders_common.c (+47 -0)
📝 src/lib_ccx/ccx_encoders_common.h (+4 -0)
src/lib_ccx/ccx_encoders_ssa.c (+173 -0)
📝 src/lib_ccx/params.c (+6 -1)
📝 src/lib_ccx/utility.c (+2 -0)

📄 Description

https://codein.withgoogle.com/dashboard/task-instances/5110381871628288/

Added basic ASS/SSA encoding support.
I checked some files, works well. Run with args: file -out=ssa or file -out=ass.
src/lib_ccx/ccx_encoders_ssa.c has two empty functions, as I can't check it without example video file, on which this functions will be called. Please give me test file if you can.
use_ass_instead_of_ssa there is, as the format is slightly different, so it is possible to generate a little different files because of different formats .ass and .ssa, but maybe we need only one format.
Please give your opinion about the ssa_header, maybe there is something to change in style.

Last pull request was closed because of bug.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/CCExtractor/ccextractor/pull/470 **Author:** [@Izaron](https://github.com/Izaron) **Created:** 12/2/2016 **Status:** ✅ Merged **Merged:** 12/2/2016 **Merged by:** [@cfsmp3](https://github.com/cfsmp3) **Base:** `master` ← **Head:** `ass-ssa-2` --- ### 📝 Commits (1) - [`f8e3788`](https://github.com/CCExtractor/ccextractor/commit/f8e378863ed590e58c208b47a446b2b811bafa47) Added basic ASS/SSA encoding support ### 📊 Changes **8 files changed** (+240 additions, -4 deletions) <details> <summary>View changed files</summary> 📝 `src/lib_ccx/ccx_common_constants.h` (+1 -0) 📝 `src/lib_ccx/ccx_common_option.h` (+1 -0) 📝 `src/lib_ccx/ccx_decoders_common.c` (+6 -3) 📝 `src/lib_ccx/ccx_encoders_common.c` (+47 -0) 📝 `src/lib_ccx/ccx_encoders_common.h` (+4 -0) ➕ `src/lib_ccx/ccx_encoders_ssa.c` (+173 -0) 📝 `src/lib_ccx/params.c` (+6 -1) 📝 `src/lib_ccx/utility.c` (+2 -0) </details> ### 📄 Description [https://codein.withgoogle.com/dashboard/task-instances/5110381871628288/](https://codein.withgoogle.com/dashboard/task-instances/5110381871628288/) Added basic ASS/SSA encoding support. I checked some files, works well. Run with args: `file -out=ssa` or `file -out=ass`. `src/lib_ccx/ccx_encoders_ssa.c` has two empty functions, as I can't check it without example video file, on which this functions will be called. Please give me test file if you can. `use_ass_instead_of_ssa` there is, as the format is slightly different, so it is possible to generate a little different files because of different formats .ass and .ssa, but maybe we need only one format. Please give your opinion about the `ssa_header`, maybe there is something to change in style. Last pull request was closed because of bug. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
claunia added the pull-request label 2026-01-29 17:00:52 +00:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/ccextractor#1287