Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Format passed to audioSink.configure is missing information. #594

Merged
merged 1 commit into from
Aug 31, 2023

Conversation

Tolriq
Copy link
Contributor

@Tolriq Tolriq commented Aug 21, 2023

AudioSink might require the identifying data and metadata to be able to take decisions.

@tonihei
Copy link
Collaborator

tonihei commented Aug 30, 2023

Thanks for the PR! Will work on merging it internally and it will then be marked as merged here too.

@tonihei tonihei self-assigned this Aug 30, 2023
AudioSink might require the identifying data and metadata to be able to take decisions.
@copybara-service copybara-service bot merged commit bbfba27 into androidx:main Aug 31, 2023
1 check passed
@Tolriq
Copy link
Contributor Author

Tolriq commented Aug 31, 2023

@tonihei thanks.
Anyway you can help me move on with #369 to generate the test asset?

While I can have a simple personal workaround extractor to address google/ExoPlayer#10807, the id3decoder issue force me to keep a full fork of the extractor library for something really simple to fix :(

@androidx androidx locked and limited conversation to collaborators Oct 31, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants