-
Notifications
You must be signed in to change notification settings - Fork 116
LAME support for mp3 format? #291
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
Comments
I think it would be best to integrate LAME into libsndfile first, and then add the (minor) modifications necessary to support it here. |
Hi @bastibe, FYI: libsndfile/libsndfile#701 (reply in thread) MP3 is now supported. |
That's fantastic news! And they now provide precompiled binaries for Windows, too! But as far as I can tell, MP3 support is not yet included in a tagged release, right? I'll update soundfile as soon as MP3 support is officially released. In the meantime, would (any of) you be interested in contributing a pull request that adds MP3 support to soundfile? |
Hi @bastibe!
|
Probably not a ton of work. Mostly adding the new Once it is confirmed working and libsndfile tags a new release, I can update the _soundfile_data repo with current binaries and release a new version of soundfile. |
Any updates here? If nobody is working on this, I might take a crack at it. |
No update. We are still waiting for libsndfile to ship their MP3 support. |
EDIT: Just saw there is already an open PR for this: #335 |
Since mp3 patents have apparently now expired (2017) I guess there isn't a legal issue with using LAME to decode mp3 files anymore? Is it worth looking into here or just waiting for libsndfile to support mp3 natively?
The text was updated successfully, but these errors were encountered: