SDL2: Crashes when using MT-32 emulation

[domino1983]

The latest Win32 daily build (October 24, 2016, 10:34 pm) seems to have problem with that game. It crashes when trying to start. It's CD/DOS/English version.
It works without problems with latest official build and it worked without problems with some older daily build (week ago).

[Kirben]

The crash is caused by SDL 2.0.5. but I'm unable to get a useful back trace via MinGW.

[OmerMor]

I tried to build from the source w/VS2015 using the wiki instructions, but it appears to be using SDL1 and not SDL2.

If you could update the instructions, I would be able to help get a stack trace.

[dreammaster]

That's correct. Although support for SDL2 has been added previously, by LordHoto I think, I'm pretty sure it does indeed still link to SDL1. Given the recent problems with SDL2, I didn't see much point in updating the project creation to use SDL2 by default yet.

You should be able to go to dists/msvc14/create_msvc14.bat, and copy either of the stable or all engines lines that call create_project, and simply add "-use-sdl2" as an extra parameter. From memory, that should allow you to generate project files that link against SDL2. As has been previously mentioned, SDL 2.0.4 DLL is being provided with the libs package, so you'd also need to get the 2.0.5 and replace the DLL.

[wjp]

I think the MT-32 emu might be calling OSD functions in the audio thread. (See also #9609)

[criezy]

If OSD is the issue then ​PR 855 should fix it. I was planning to merge it into master this weekend (after taking the time to look at the comment from fuzzie) but not into the 1.9 branch since cloud is not enabled there anyway. But if this fixes this bug I will revise that plan.

[Kirben]

I tried adding ​PR 855, but the crash still occurs.

[eriktorbjorn]

There's also this SDL bug: ​https://bugzilla.libsdl.org/show_bug.cgi?id=3584

So presumably it will work with their next release, whenever that is.

[wjp]

Munt 2.0.3 also reduces its stack footprint ( ​https://sourceforge.net/p/munt/bugs/28/ ). I've now updated our internal version to this, so hopefully this is now resolved, even with SDL-2.0.5.

[csnover]

I have not seen any reports of this being a problem since the Munt update, so I’m going to mark this ticket as resolved for now. Please feel free to comment or reopen if anything changes.