Fixing a couple of flaky tests, a missed change for MIDI time adjustment, as well fixing voice looping that causes notes to end too early.
--
v3: dmime: Force recompute MIDI message reference time.
dmsynth: Set loop and sample generators on the fluid_voice.
dmusic/tests: Skip test_default_gm_collection if gm.dls is missing.
dmime/tests: Ignore failure on missing gm.dls in test_band_track_play.
dmime/tests: Avoid checking message segment state reference count.
dmime/tests: Remove flaky track playing state test.
https://gitlab.winehq.org/wine/wine/-/merge_requests/4176
Fixing a couple of flaky tests, a missed change for MIDI time adjustment, as well fixing voice looping that causes notes to end too early.
--
v2: dmime: Force recompute MIDI message reference time.
dmsynth: Set loop and sample generators on the fluid_voice.
dmusic/tests: Skip test_default_gm_collection if gm.dls is missing.
dmime/tests: Ignore failure on missing gm.dls in test_band_track_play.
dmime/tests: Avoid checking message segment state reference count.
dmime/tests: Remove flaky track playing state test.
https://gitlab.winehq.org/wine/wine/-/merge_requests/4176
Single float division (divss) is at least four time slower,
than multiplication (mulss).
More information: https://www.agner.org/optimize/instruction_tables.pdf
The units_to_pixels was optimized to use only multiplication (mulss),
and avoid using divisions (divss).
![Screenshot_2023-10-25_at_12.15.14](/uploads/4db35ba503b97b070ac20f35e6456445/Screenshot_2023-10-25_at_12.15.14.png)
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/4177
This seems to be the place of this flag on Windows. I am attaching the test here which suggests that this is the place for the flag. I am not including is in MR as afraid it may be flaky (although currently succeeded on all the testbot machines), as many factors may affect if the block actually ends up LFH or not.
The motivation under doing this is that it should help "Binding of Isaac: Rebirth" to avoid random crashes, by making `*(int *)((int *)allocated_ptr_of_size_4 - 1)` negative, which, due to some checks on the value, avoids further (potentially fatal) out of bound memory access.
[lfh_flag_test.patch](/uploads/0048642b107783f5758320e06cd3d629/lfh_flag_test.patch)
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/4145