[PR #1819] style: use CCX_STREAM_TYPE_VIDEO_HEVC enum instead of raw 0x24 #2567

New Issue

claunia · 2026-01-29T17:22:48Z

claunia commented

2026-01-29 17:22:48 +00:00

Original Pull Request: https://github.com/CCExtractor/ccextractor/pull/1819

State: closed
Merged: Yes

Summary

Follow-up to PR #1769 - use the defined enum constant CCX_STREAM_TYPE_VIDEO_HEVC instead of the magic number 0x24 for better code maintainability and consistency.

Changes

Replace 0x24 with CCX_STREAM_TYPE_VIDEO_HEVC in PMT parsing condition
Simplify get_printable_stream_type() case statement by removing redundant assignment (the enum value passes through unchanged)
Update comment to list all three supported video stream types

Testing

Tested with ATSC 3.0 sample from issue #1639 - HEVC streams are still properly recognized.

🤖 Generated with Claude Code

**Original Pull Request:** https://github.com/CCExtractor/ccextractor/pull/1819 **State:** closed **Merged:** Yes --- ## Summary Follow-up to PR #1769 - use the defined enum constant `CCX_STREAM_TYPE_VIDEO_HEVC` instead of the magic number `0x24` for better code maintainability and consistency. ## Changes - Replace `0x24` with `CCX_STREAM_TYPE_VIDEO_HEVC` in PMT parsing condition - Simplify `get_printable_stream_type()` case statement by removing redundant assignment (the enum value passes through unchanged) - Update comment to list all three supported video stream types ## Testing Tested with ATSC 3.0 sample from issue #1639 - HEVC streams are still properly recognized. 🤖 Generated with [Claude Code](https://claude.com/claude-code)

claunia added the pull-request label 2026-01-29 17:22:48 +00:00

claunia closed this issue

2026-01-29 17:22:48 +00:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/ccextractor#2567