Mix_LoadMUS_RW in SDL_mixer 1.2.4

Is there any particular reason why no ‘Mix_LoadMUS_RW’ is not included
in SDL_mixer 1.2.4? It seems to me that this would be trivial to
implement (for MikMod at least; I don’t care about other formats).–
Rainer Deyke | root at rainerdeyke.com | http://rainerdeyke.com

Is there any particular reason why no ‘Mix_LoadMUS_RW’ is not included
in SDL_mixer 1.2.4? It seems to me that this would be trivial to
implement (for MikMod at least; I don’t care about other formats).

You don’t look busy; send a patch. :slight_smile:

–ryan.

You don’t look busy; send a patch. :slight_smile:

Specifically, from SDL_mixer.h:

#if 0 /* This hasn’t been hooked into music.c yet /
/
Load a music file from an SDL_RWop object (MikMod-specific currently)
Matt Campbell (matt at campbellhome.dhs.org) April 2000 */
extern no_parse_DECLSPEC Mix_Music * SDLCALL Mix_LoadMUS_RW(SDL_RWops *rw);
#endif

In case you haven’t seen that. It’s probably a mixture of developer
inertia and the fact that adding an API call just for mikmod is a little
yucky.

–ryan.

Ryan C. Gordon wrote:

#if 0 /* This hasn’t been hooked into music.c yet /
/
Load a music file from an SDL_RWop object (MikMod-specific currently)
Matt Campbell (matt at campbellhome.dhs.org) April 2000 */
extern no_parse_DECLSPEC Mix_Music * SDLCALL Mix_LoadMUS_RW(SDL_RWops *rw);
#endif

In case you haven’t seen that. It’s probably a mixture of developer
inertia and the fact that adding an API call just for mikmod is a little
yucky.

although i would guess that MP3 would also work, since SMPEG can already
deal directly with SDL_RWops pointers :]
(not that i’m volunteering anything, heh)