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.

Please don't commit this right now. I'll be reworking it due to:

http://lists.libsdl.org/pipermail/sdl-libsdl.org/2011-July/081439.html

Created attachment 654 [details]
map_file_api_to_rwops.patch

OK, here's a patch that uses the new Mix_LoadMUSType_RW() function in order to clean up the filename-based loading functions:

Map filename-based Mix_Load* functions to RWops-based ones

Mix_LoadMUS() and Mix_LoadWAV(), which take a filename argument, pretty much
duplicate some functionality of what the RWops-based functions do. This patch
cleans some of this up, using the RWops-based functions wherever possible.

The following functions have been deleted entirely, as they're no longer needed:

  SMPEG_new(), FLAC_new(), map_openFile(), MOD_new(), modplug_new(), OGG_new(),
  native_midi_loadsong(), Timidity_LoadSong(), WAVStream_LoadSong().

In order to avoid code duplication when detecting the format of the loaded
files, two new functions were introduced. An internal helper function:

  static Mix_MusicType detect_music_type(SDL_RWops *rw)

and a new API function:

  Mix_Music *Mix_LoadMUSType_RW(SDL_RWops *rw, Mix_MusicType type)

The new function allows to override SDL_mixer's auto-detection of the
music format. Mix_LoadMUS_RW() is now a simple wrapper for
Mix_LoadMUSType_RW(). Mix_LoadMUS() uses this in order to guess the file format
from the file's extension.

This patch should preserve source and binary backwards compatibility, as well
as behavior. (And that means that RWops leaks that occured before will still
occur; this patch doesn't fix those.)

Created attachment 655 [details]
fix_stray_else.patch

Oopsy, there's a stray "else" in the code that can mess with Mix_LoadMUS(). Here's a small followup patch.

Bumping priority on a few SDL_mixer bugs. This is just so we know to look at them for an upcoming release, but once we look at them more closely, we may decide to flag them as WONTFIX or push them back to a later release, so don't take this change in priority as any promise of anything, yet.  :)

--ryan.

I like the idea of this patch, but I just fixed the music memory leak and this patch doesn't apply anymore.

Could you create a new version of this patch based on the latest source available here?
http://hg.libsdl.org/SDL_mixer

If you need a snapshot archive, let me know and I can arrange that.

Thanks!

Created attachment 756 [details]
map_file_api_to_rwops.patch

(In reply to comment #5)
> Could you create a new version of this patch based on the latest source
> available here?
> http://hg.libsdl.org/SDL_mixer

I rebased it to current Hg.  Attaching.

This build error:

music.c: In function 'music_halt_or_loop':
music.c:203:41: error: 'native_midi_ok' undeclared (first use in this function)
music.c:203:41: note: each undeclared identifier is reported only once for each function it appears in

Was there before, it's not due to this patch.

I just did a fresh build of SDL_mixer and didn't get any errors.  Are you're sure it's not due to local changes?

(In reply to comment #7)
> I just did a fresh build of SDL_mixer and didn't get any errors.  Are you're
> sure it's not due to local changes?

Yes.  The error is there with a fresh clone from hg.  I just opened bug 1358 for this.

Okay, I reviewed the patch and it looks good.
I made a few tweaks, including adding a freesrc parameter, looking at the file extension first, and greatly improving the MP3 detection.

Thanks!
http://hg.libsdl.org/SDL_mixer/rev/3de4970b36d4