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

OptimFROG: support encoder version >= 5.100 #406

Merged
merged 2 commits into from Oct 24, 2019

Conversation

@phw
Copy link
Contributor

phw commented Oct 21, 2019

Encoder 5.100 allows optional additional fields in header, which the current code does not expect. Currently it expects a header data size of exactly 12 or 15 bytes, but the latest spec allow for arbitrary longer size with X additional bytes, where X is the indicated data size - 15 (the fixed fields before).

Also added extraction of bits per sample and encoder version information.

See spec in formats.txt and changelog.txt from downloads in http://losslessaudio.org/Downloads.php

Also extract bits per sample and encoder version information.
@phw phw force-pushed the phw:fix/optimfrog-5100 branch from 44d35a1 to 7171f47 Oct 21, 2019
@phw phw changed the title optimfrog: support encoder version >= 5.100 OptimFROG: support encoder version >= 5.100 Oct 21, 2019
@phw phw force-pushed the phw:fix/optimfrog-5100 branch from 87aa03f to 4815741 Oct 23, 2019
@lazka lazka merged commit 2015790 into quodlibet:master Oct 24, 2019
2 checks passed
2 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
quodlibet.mutagen #20191023.12 succeeded
Details
@phw phw deleted the phw:fix/optimfrog-5100 branch Oct 24, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.