Skip to content

Conversation

@NivinCNC
Copy link
Contributor

@NivinCNC NivinCNC commented Dec 2, 2025

Introduced the AudioFile data class and related API for handling external audio tracks. Updated ExtractorLink and player logic to support merging separate audio tracks with video playback, allowing for custom audio sources and headers.

Introduced the AudioFile data class and related API for handling external audio tracks. Updated ExtractorLink and player logic to support merging separate audio tracks with video playback, allowing for custom audio sources and headers.
Annotated the AudioFile data class with @ConsistentCopyVisibility to ensure consistent visibility behavior, likely for improved API usage or codebase consistency.
Copy link
Collaborator

@fire-light42 fire-light42 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The code looks good, but I have not tested yet.

Replaces manual MediaItem construction with getMediaItem for audio tracks and simplifies header handling by using a local variable. This improves code clarity and consistency in media item creation.
@NivinCNC
Copy link
Contributor Author

NivinCNC commented Dec 4, 2025

Refactored.

@NivinCNC NivinCNC requested a review from fire-light42 December 4, 2025 12:12
Added the @prerelease annotation to the AudioFile data class and the newAudioFile function to indicate their prerelease status.
@NivinCNC
Copy link
Contributor Author

NivinCNC commented Dec 5, 2025

Added @prerelease Annotation

@NivinCNC NivinCNC requested a review from Luna712 December 5, 2025 05:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants