We are currently migrating Bugzilla to GitHub issues.
Any changes made to the bug tracker now will be lost, so please do not post new bugs or make changes to them.
When we're done, all bug URLs will redirect to their equivalent location on the new bug tracker.

Bug 2985 - mp3 and flac support in LoadWav_RW
Summary: mp3 and flac support in LoadWav_RW
Status: RESOLVED FIXED
Alias: None
Product: SDL_mixer
Classification: Unclassified
Component: misc (show other bugs)
Version: 2.0.0
Hardware: All All
: P2 enhancement
Assignee: Ryan C. Gordon
QA Contact: Sam Lantinga
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-05-16 01:23 UTC by muellppb
Modified: 2015-06-09 07:44 UTC (History)
1 user (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description muellppb 2015-05-16 01:23:17 UTC 
I think it would be nice to have mp3 and flac support for LoadWav_RW or is there a particular reason that I'm not aware of? 

I even found a (the?) GitHub-Project for a Blackberry OS10 port where mp3 and flac support were added (and, as I could see from skimming through the patch, in a platform-independant way). The repository in question is: https://github.com/drodin/SDL2_mixer

So I think it _probably_ (one never knows) wouldn't be much work to adapt the relevant patches to get them to work with the current SDL2_mixer-Code. 

I even volunteer to test (well, at least for Windows and Linux) and adapt the patches if necessary.

(Side-note concerning flac-support: The Documentation says "no", but it is mentioned in Bug #543 and I haven't tried it yet)
Comment 1 Philipp Wiesemann 2015-05-21 20:26:53 UTC 
Mix_LoadWAV_RW() already supports FLAC and MP3.

See here:
https://hg.libsdl.org/SDL_mixer/file/25605697981c/mixer.c#l584
Comment 2 muellppb 2015-06-09 07:44:20 UTC 
I haven't had time to test it, so I close this ticket for now and open up another if there are bugs.