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

Fix empty sample buffers returned when seeking FLAC files. #173

Merged
merged 3 commits into from
Dec 3, 2022

Conversation

psobot
Copy link
Member

@psobot psobot commented Dec 2, 2022

Fixes #170.

See juce-framework/JUCE#1157 for the root cause. This PR vendors the juce::FlacAudioFormat classes to manually patch out two lines of code that were causing the issue.

A nearby bug was also fixed: if reading a PythonInputStream to its end, all subsequent calls to PythonInputStream::isExhausted would continue to return true, even if that stream had been seeked back to an earlier point.

@psobot psobot added the bug Something isn't working label Dec 2, 2022
@psobot psobot force-pushed the psobot/flac-silence branch from 705091e to cd4c60f Compare December 2, 2022 14:33
@psobot psobot force-pushed the psobot/flac-silence branch from cd4c60f to 95fd164 Compare December 2, 2022 20:06
@psobot psobot merged commit 872d332 into master Dec 3, 2022
@psobot psobot deleted the psobot/flac-silence branch December 3, 2022 22:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Flac files return empty sample buffers
1 participant