[PR #1832] [CLOSED] [FIX] Prevent creation of empty per-stream subtitle files #2587

Open
opened 2026-01-29 17:22:57 +00:00 by claunia · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/CCExtractor/ccextractor/pull/1832
Author: @VyankateshD206
Created: 12/15/2025
Status: Closed

Base: masterHead: fix-empty-output-files-issue-1282


📝 Commits (1)

  • 71c81d4 [FIX] Prevent creation of empty per-stream subtitle files

📊 Changes

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

View changed files

📝 src/lib_ccx/ccx_encoders_common.c (+22 -3)
📝 src/lib_ccx/ccx_encoders_structs.h (+1 -0)

📄 Description

In raising this pull request, I confirm the following (please check boxes):

  • 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.
  • I have mentioned this change in the changelog.

My familiarity with the project is as follows (check one):

  • 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.

[FIX] Prevent creation of empty per-stream subtitle files (Fixes #1282)

This pull request implements deferred output file creation to avoid producing zero-byte .srt files for caption streams that contain no actual data.

  • Track per-output writes via wrote_data on ccx_s_write.
  • When no data is written to an output, skip footer writing and delete the file on teardown to avoid empty cc3 artifacts.

🔄 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/1832 **Author:** [@VyankateshD206](https://github.com/VyankateshD206) **Created:** 12/15/2025 **Status:** ❌ Closed **Base:** `master` ← **Head:** `fix-empty-output-files-issue-1282` --- ### 📝 Commits (1) - [`71c81d4`](https://github.com/CCExtractor/ccextractor/commit/71c81d40290f36a336fab9154d631fb646ec05d2) [FIX] Prevent creation of empty per-stream subtitle files ### 📊 Changes **2 files changed** (+23 additions, -3 deletions) <details> <summary>View changed files</summary> 📝 `src/lib_ccx/ccx_encoders_common.c` (+22 -3) 📝 `src/lib_ccx/ccx_encoders_structs.h` (+1 -0) </details> ### 📄 Description <!-- Please prefix your pull request with one of the following: **[FEATURE]** **[FIX]** **[IMPROVEMENT]**. --> **In raising this pull request, I confirm the following (please check boxes):** - [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. - [X] **I have mentioned this change in the [changelog](https://github.com/CCExtractor/ccextractor/blob/master/docs/CHANGES.TXT).** **My familiarity with the project is as follows (check one):** - [ ] 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. --- ## **[FIX] Prevent creation of empty per-stream subtitle files (Fixes #1282)** #### This pull request implements deferred output file creation to avoid producing zero-byte `.srt` files for caption streams that contain no actual data. - Track per-output writes via wrote_data on ccx_s_write. - When no data is written to an output, skip footer writing and delete the file on teardown to avoid empty cc3 artifacts. --- <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:22:57 +00:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/ccextractor#2587