[PR #729] [MERGED] [FEATURE] Add WebVTT output from Matroska (Issue #724) #1551

New Issue

claunia · 2026-01-29T17:17:09Z

claunia commented

2026-01-29 17:17:09 +00:00

📋 Pull Request Information

Original PR: https://github.com/CCExtractor/ccextractor/pull/729
Author: @LucasYoung
Created: 3/30/2017
Status: ✅ Merged
Merged: 3/31/2017
Merged by: @cfsmp3

Base: master ← Head: master

📝 Commits (2)

dd5c1ee Added WebVTT output from Matroska
5ffa442 Removed changes to ccextractor.vcxproj

📊 Changes

3 files changed (+139 additions, -2 deletions)

View changed files

📝 src/lib_ccx/matroska.c (+117 -2)
📝 src/lib_ccx/matroska.h (+13 -0)
📝 src/lib_ccx/utility.c (+9 -0)

📄 Description

In raising this pull request, I confirm the following:

I have read and understood the contributors guide.
I have checked that another pull request for this purpose does not exist.
I have considered, and confirmed that this submission will be valuable to others.
I accept that this submission may not be used, and the pull request closed at the will of the maintainer.
I give this submission freely, and claim no ownership to its content.

My familiarity with the project is as follows:

I have never used CCExtractor.
I have used CCExtractor just a couple of times.
I absolutely love CCExtractor, but have not contributed previously.
I am an active contributor to CCExtractor.

This pull request implements extraction of WebVTT subtitles from Matroska files. I am hoping to earn 5 GSOC qualification points from this pull request.

I have tested this code with the sample video in issue #724 and the output appears to be correct. However, I have not been able to find other applicable test cases for this code (mkv files with WebVTT subtitles). Also, all of my testing was done on a 64-bit Windows machine.

_{🔄 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/729 **Author:** [@LucasYoung](https://github.com/LucasYoung) **Created:** 3/30/2017 **Status:** ✅ Merged **Merged:** 3/31/2017 **Merged by:** [@cfsmp3](https://github.com/cfsmp3) **Base:** `master` ← **Head:** `master` --- ### 📝 Commits (2) - [`dd5c1ee`](https://github.com/CCExtractor/ccextractor/commit/dd5c1ee243fea59e63a5cea38d40637095f25bed) Added WebVTT output from Matroska - [`5ffa442`](https://github.com/CCExtractor/ccextractor/commit/5ffa442c5f08b78f517d5699b1175ebaa6b32202) Removed changes to ccextractor.vcxproj ### 📊 Changes **3 files changed** (+139 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `src/lib_ccx/matroska.c` (+117 -2) 📝 `src/lib_ccx/matroska.h` (+13 -0) 📝 `src/lib_ccx/utility.c` (+9 -0) </details> ### 📄 Description **In raising this pull request, I confirm the following:** - [X] I have read and understood the [contributors guide](https://github.com/CCExtractor/ccextractor/blob/master/.github/CONTRIBUTING.md). - [X] I have checked that another pull request for this purpose does not exist. - [X] I have considered, and confirmed that this submission will be valuable to others. - [X] I accept that this submission may not be used, and the pull request closed at the will of the maintainer. - [X] I give this submission freely, and claim no ownership to its content. **My familiarity with the project is as follows:** - [ ] I have never used CCExtractor. - [X] I have used CCExtractor just a couple of times. - [ ] I absolutely love CCExtractor, but have not contributed previously. - [ ] I am an active contributor to CCExtractor. --- This pull request implements extraction of WebVTT subtitles from Matroska files. I am hoping to earn 5 GSOC qualification points from this pull request. I have tested this code with the sample video in issue #724 and the output appears to be correct. However, I have not been able to find other applicable test cases for this code (mkv files with WebVTT subtitles). Also, all of my testing was done on a 64-bit Windows machine. --- <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:17:09 +00:00

claunia closed this issue