[PR #1914] [MERGED] [FIX] Resolve unused variable warnings in ts_tables_epg.c #2708

New Issue

claunia · 2026-01-29T17:23:32Z

claunia commented

2026-01-29 17:23:32 +00:00

📋 Pull Request Information

Original PR: https://github.com/CCExtractor/ccextractor/pull/1914
Author: @VS7686
Created: 12/27/2025
Status: ✅ Merged
Merged: 12/29/2025
Merged by: @cfsmp3

Base: master ← Head: fix-epg-warnings

📝 Commits (3)

00d8c9c Fix unused variable warnings in ts_tables_epg.c
d4a7b1d Trigger CI re-run
29c3f4e Trigger CI re-run 2

📊 Changes

1 file changed (+1 additions, -3 deletions)

View changed files

📝 src/lib_ccx/ts_tables_epg.c (+1 -3)

📄 Description

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

[x ] I have read and understood the contributors guide.
[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.

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.

---### Description
This PR fixes compiler warnings in src/lib_ccx/ts_tables_epg.c related to unused variables.

The Issue:
The variables mode and compression_type were declared and assigned values from the offset array, but they were never subsequently used in the logic block (lines 98-105). This triggered -Wunused-but-set-variable warnings during the build.

The Fix:

Removed the declarations and assignments for mode and compression_type.
Preserved the critical offset += 3; pointer arithmetic to ensure the parsing logic remains correct.
Verified that the code builds without these specific warnings.

Environment:

Windows 11
MSVC / CMake

_{🔄 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/1914 **Author:** [@VS7686](https://github.com/VS7686) **Created:** 12/27/2025 **Status:** ✅ Merged **Merged:** 12/29/2025 **Merged by:** [@cfsmp3](https://github.com/cfsmp3) **Base:** `master` ← **Head:** `fix-epg-warnings` --- ### 📝 Commits (3) - [`00d8c9c`](https://github.com/CCExtractor/ccextractor/commit/00d8c9cb0af96637c7cb20d0770cba17f935ecbc) Fix unused variable warnings in ts_tables_epg.c - [`d4a7b1d`](https://github.com/CCExtractor/ccextractor/commit/d4a7b1d6edd80695cce18e536ae600f36426cbba) Trigger CI re-run - [`29c3f4e`](https://github.com/CCExtractor/ccextractor/commit/29c3f4e68495561cedf4a124349289525d7d8fcd) Trigger CI re-run 2 ### 📊 Changes **1 file changed** (+1 additions, -3 deletions) <details> <summary>View changed files</summary> 📝 `src/lib_ccx/ts_tables_epg.c` (+1 -3) </details> ### 📄 Description  **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. ---### Description This PR fixes compiler warnings in `src/lib_ccx/ts_tables_epg.c` related to unused variables. **The Issue:** The variables `mode` and `compression_type` were declared and assigned values from the `offset` array, but they were never subsequently used in the logic block (lines 98-105). This triggered `-Wunused-but-set-variable` warnings during the build. **The Fix:** - Removed the declarations and assignments for `mode` and `compression_type`. - Preserved the critical `offset += 3;` pointer arithmetic to ensure the parsing logic remains correct. - Verified that the code builds without these specific warnings. **Environment:** - Windows 11 - MSVC / CMake --- <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:23:32 +00:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/ccextractor#2708