Özkan Sezer

Results 570 comments of Özkan Sezer

Fixed by PR #44. Closing the ticket.

This shouldn't be fixed by 35488c6, it should be #248 instead.

Knowing the SDL2 version in use in each case _might_ help understand things.

> * There seems to be some bad interaction between sdl12_compat > and SDL_Mixer, or I did something wrong. The following lines keep > triggering a heap corruption error: >...

> Commit https://github.com/libsdl-org/sdl12-compat/commit/cc23a07dc4c7a720d192285cc38d1bd90508f72d can cause a crash, as there can be > MOUSEMOTION events before VideoSurface12 is initialized. So: something like the following? (CC: @sulix) ```diff diff --git a/src/SDL12_compat.c b/src/SDL12_compat.c...

> Should we just drop the event if there's no VideoSurface12 yet? Well, why not.

OK, SDL_mixer-1.2 is possibly using `free()` for `SDL_free` here. Which compiler are SDL2 (and your SDL_mixer-1.2) built with? Specifically: is SDL2 built with Visual Studio? Problem is, the audio file...

> If my assumptions are correct, what is the solution here? One thing I can think of: rebuilding SDL_mixer-1.2 specifically against sdl12-compat: In sdl12-compat, `SDL_free` is a function call and...

> If my assumptions are correct, what is the solution here? OR: a better solution would be calling SDL_FreeWAV for loaded wav files in SDL_mixer-1.2: Does the following patch fix...

Better, more complete patch for SDL_mixer-1.2: ```diff diff --git a/mixer.c b/mixer.c index 431f6e6..bae3271 100644 --- a/mixer.c +++ b/mixer.c @@ -560,6 +560,7 @@ Mix_Chunk *Mix_LoadWAV_RW(SDL_RWops *src, int freesrc) SDL_AudioSpec wavespec, *loaded;...