mirror of
https://github.com/ZDoom/fluidsynth.git
synced 2025-05-29 16:31:19 +00:00
44453ff232
The parameters (roomsize, level, etc.) of the reverb effects unit are initialized at the very end of `new_fluid_synth()` with `fluid_synth_set_reverb_full_LOCAL()`. This however only adds an update-event to the `rvoice_mixer` queue. The call to `fluid_synth_process_event_queue()` in `new_fluid_synth()` should make sure, that this event is dispatched and triggers the actual update. However, the event is not dispatched immediately, because the rvoice event queue has not been committed yet, that is, a call to `fluid_rvoice_eventhandler_flush()` is missing. So, although a reverb param initialization event has been queued, the reverb params still are garbage initialized (on Windows to some `-6.2774385622041925e+66`). The next call to through the synth's public API will flush the queue and finally dispatch the update event, but when will it happen? 1. If the soundfont is specified as command-line argument, this call will happen before the audio driver starts rendering. 2. If the soundfont is loaded via shell command `load`, the audio driver will first start rendering audio, after updating the reverb. Case 1. is trivial, everything works as it should. Case 2. is interesting. Since the synth already started rendering audio by using that uninitialized reverb unit, the reverb engine's internal buffer is completely filled up with noise. Before outputting that signal to sound card, the sound is clipped to `1.0f`. That's the click we hear at the beginning. And because the reverb is so loud, the rendered audio signal stays 1.0f for quite a long time (...or always, can't tell). Why is it not reproducible on Linux? Because GCC and Clang (AFAIK) leave all values uninitialized after allocating memory. And because malloc() often return zero-initialized memory, the reverb params seem to be nicely zero-initialized. MSVC however, always initializes memory with garbage, which is why we "hear" this resonance disaster. Solution: Just update the reverb params via public API, which implicitly calls `fluid_rvoice_eventhandler_flush()`. Fixes #563. |
||
|---|---|---|
| .. | ||
| bindings | ||
| drivers | ||
| gentables | ||
| midi | ||
| rvoice | ||
| sfloader | ||
| synth | ||
| utils | ||
| CMakeLists.txt | ||
| config.cmake | ||
| fluidsynth.c | ||