[SDL_mixer] Mapping file-based API to RWops

Before I start doing the work, it’s better to ask first:

Currently, functions like Mix_LoadMUS() and Mix_LoadWAV() which take a
filename pretty much duplicate a lot of what RWops-based functions do,
which take an RWops as parameter.

I’d like to go ahead and remove most of the duplicate code and make all
filename-based functions map to the RWops-based ones. For example,
Mix_LoadMUS() would open an RWops on the filename and then call
Mix_LoadMUS_RW() on the RWops.

Can anyone imagine possible problems with that? Should I go ahead and
start working on it?

Can anyone imagine possible problems with that? Should I go ahead and
start working on it?

Oh my god, yes, please fix that.

–ryan.

I now posted a patch in bugzilla:

http://bugzilla.libsdl.org/show_bug.cgi?id=1252

Please let me know about anything that might need changes/reworking.On 07/24/2011 09:27 AM, Ryan C. Gordon wrote:

Can anyone imagine possible problems with that? Should I go ahead and
start working on it?

Oh my god, yes, please fix that.