style: use CCX_STREAM_TYPE_VIDEO_HEVC enum instead of raw 0x24 #1819
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Follow-up to PR #1769 - use the defined enum constant for HEVC stream type (0x24) instead of magic numbers for better code maintainability. Also simplifies the case statement in get_printable_stream_type() by removing redundant assignment since the enum value passes through unchanged. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Follow-up to PR #1769 - use the defined enum constant
CCX_STREAM_TYPE_VIDEO_HEVCinstead of the magic number0x24for better code maintainability and consistency.Changes
0x24withCCX_STREAM_TYPE_VIDEO_HEVCin PMT parsing conditionget_printable_stream_type()case statement by removing redundant assignment (the enum value passes through unchanged)Testing
Tested with ATSC 3.0 sample from issue #1639 - HEVC streams are still properly recognized.
🤖 Generated with Claude Code