Özkan Sezer

Patch reverted from both SDL2 and main branches. That reverted the `FLUIDSYNTH_Pause` and `FLUIDSYNTH_Resume` additions too: if we do really want them, one should hand-pick them and put them back.

I can't reproduce this on linux at the moment. I added this debug print: ```diff diff --git a/dynamic_mod.c b/dynamic_mod.c index e184ee7..a5a4388 100644 --- a/dynamic_mod.c +++ b/dynamic_mod.c @@ -219,6 +219,7 @@...

With stb_vorbis backend, it plays. With libvorbis backend: reproduced the issue: the `playmus` program with an `-r` option anything _smaller than 96000_ just quits: ``` $ ./playmus -r 48000 ~/3/1/adv_zenhan.ogg...

The built-in timidity is a fork of an old timidity version, namely 0.2i, therefore it doesn't have soundfont support.

There is an initial patchset for some soundfont support, though. See here: https://github.com/sezero/libtimidity/compare/master...soundfont AFAIR, the missing thing there is the lack of parsing timidity++ style cfg files with both gus...

Better after the latest changes. I suggest the following to accomodate @slouken's arguments at https://github.com/libsdl-org/SDL_ttf/pull/312#discussion_r1549876730: it makes it safer, just in case. Also changes some error returns to simply return...

This looks fair to me now. @1bsyl, @slouken: what do you think?

The issue seems to happen in qs too: https://github.com/sezero/quakespasm/issues/53 .. but only in rerelease start map. Does this actually happen anywhere else?

> @sezero Seems like a new vis patch for the rerelease data would work? Really don't know, never ever tried it

So, it's a map-specific issue and not worth caring about much, yes?