From: Anton Baskanov baskanov@gmail.com
--- dlls/dmsynth/synth.c | 34 ++++++++++++++++++++++++++-------- dlls/dmsynth/tests/dmsynth.c | 10 +++++----- 2 files changed, 31 insertions(+), 13 deletions(-)
diff --git a/dlls/dmsynth/synth.c b/dlls/dmsynth/synth.c index d6d633429e2..24c1b8a2201 100644 --- a/dlls/dmsynth/synth.c +++ b/dlls/dmsynth/synth.c @@ -469,6 +469,8 @@ static void synth_reset_default_values(struct synth *This)
for (chan = 0; chan < 0x10; chan++) { + fluid_synth_program_select(This->fluid_synth, chan, fluid_sfont_get_id(This->fluid_sfont), 0, 0); + fluid_synth_cc(This->fluid_synth, chan | 0xe0 /* PITCH_BEND */, 0, 0); fluid_synth_cc(This->fluid_synth, chan | 0xd0 /* CHANNEL_PRESSURE */, 0, 0);
@@ -1139,6 +1141,7 @@ static HRESULT WINAPI synth_Render(IDirectMusicSynth8 *iface, short *buffer, { BYTE status = event->midi[0] & 0xf0, chan = event->midi[0] & 0x0f; LONGLONG offset = event->position - position; + int sfont_id, bank_num, preset_num;
if (offset >= length) break; if (offset > 0) @@ -1163,9 +1166,17 @@ static HRESULT WINAPI synth_Render(IDirectMusicSynth8 *iface, short *buffer, break; case MIDI_CONTROL_CHANGE: fluid_synth_cc(This->fluid_synth, chan, event->midi[1], event->midi[2]); + if (event->midi[1] == MIDI_CC_BANK_LSB || event->midi[1] == MIDI_CC_BANK_MSB) + { + int bank_lsb, bank_msb; + fluid_synth_get_cc(This->fluid_synth, chan, MIDI_CC_BANK_LSB, &bank_lsb); + fluid_synth_get_cc(This->fluid_synth, chan, MIDI_CC_BANK_MSB, &bank_msb); + fluid_synth_bank_select(This->fluid_synth, chan, bank_lsb | (bank_msb << 7)); + } break; case MIDI_PROGRAM_CHANGE: - fluid_synth_program_change(This->fluid_synth, chan, event->midi[1]); + fluid_synth_get_program(This->fluid_synth, chan, &sfont_id, &bank_num, &preset_num); + fluid_synth_program_select(This->fluid_synth, chan, sfont_id, bank_num, event->midi[1]); break; case MIDI_PITCH_BEND_CHANGE: fluid_synth_pitch_bend(This->fluid_synth, chan, event->midi[1] | (event->midi[2] << 7)); @@ -1505,7 +1516,7 @@ static int synth_preset_get_num(fluid_preset_t *fluid_preset) return preset->patch; }
-static void find_region(struct synth *synth, int bank, int patch, int key, int vel, +static void find_region(struct synth *synth, int patch, int key, int vel, struct instrument **out_instrument, struct region **out_region) { struct instrument *instrument; @@ -1516,8 +1527,8 @@ static void find_region(struct synth *synth, int bank, int patch, int key, int v
LIST_FOR_EACH_ENTRY(instrument, &synth->instruments, struct instrument, entry) { - if (bank == 128 && instrument->patch == (0x80000000 | patch)) break; - else if (instrument->patch == ((bank << 8) | patch)) break; + if (instrument->patch == patch) + break; }
if (&instrument->entry == &synth->instruments) @@ -1887,18 +1898,25 @@ static int synth_preset_noteon(fluid_preset_t *fluid_preset, fluid_synth_t *flui struct region *region; struct voice *voice; struct wave *wave; + UINT patch;
TRACE("(%p, %p, %u, %u, %u)\n", fluid_preset, fluid_synth, chan, key, vel);
+ patch = preset->patch; + patch |= (preset->bank << 8) & 0x007f00; + patch |= (preset->bank << 9) & 0x7f0000; + if (chan == 9) + patch |= 0x80000000; + EnterCriticalSection(&synth->cs);
- find_region(synth, preset->bank, preset->patch, key, vel, &instrument, ®ion); - if (!region && preset->bank == 128) - find_region(synth, preset->bank, 0, key, vel, &instrument, ®ion); + find_region(synth, patch, key, vel, &instrument, ®ion); + if (!region && chan == 9) + find_region(synth, patch & 0x80000000, key, vel, &instrument, ®ion);
if (!instrument) { - WARN("Could not find instrument with patch %#x\n", preset->patch); + WARN("Could not find instrument with bank %#x, patch %#x\n", preset->bank, preset->patch); LeaveCriticalSection(&synth->cs); return FLUID_FAILED; } diff --git a/dlls/dmsynth/tests/dmsynth.c b/dlls/dmsynth/tests/dmsynth.c index 55651922c00..aa3c922c115 100644 --- a/dlls/dmsynth/tests/dmsynth.c +++ b/dlls/dmsynth/tests/dmsynth.c @@ -1927,7 +1927,7 @@ static void test_instrument_selection(void) midi.messages[1] = make_program_change(0, 0, 0); midi.messages[2] = default_note_on; midi.messages[3] = default_note_off; - check_volume_envelope(synth, &download, &midi, &default_volume_envelope, TRUE); + check_volume_envelope(synth, &download, &midi, &default_volume_envelope, FALSE);
/* there is no bank fallback on melodic channels */ memset(&midi, 0, sizeof(midi)); @@ -1937,7 +1937,7 @@ static void test_instrument_selection(void) midi.messages[3] = default_note_off; envelope = default_volume_envelope; envelope.gain = -960.; - check_volume_envelope(synth, &default_instrument_download, &midi, &envelope, TRUE); + check_volume_envelope(synth, &default_instrument_download, &midi, &envelope, FALSE);
/* bank select LSB corresponds to the patch number bits 8..14 */ download = default_instrument_download; @@ -1947,7 +1947,7 @@ static void test_instrument_selection(void) midi.messages[1] = make_program_change(0, 0, 0); midi.messages[2] = default_note_on; midi.messages[3] = default_note_off; - check_volume_envelope(synth, &download, &midi, &default_volume_envelope, TRUE); + check_volume_envelope(synth, &download, &midi, &default_volume_envelope, FALSE);
/* drum channel */
@@ -1997,7 +1997,7 @@ static void test_instrument_selection(void) midi.messages[2] = make_program_change(0, 9, 0); midi.messages[3] = make_note_on(0, 9, 60, 127); midi.messages[4] = make_note_off(10000000, 9, 60, 127); - check_volume_envelope(synth, &download, &midi, &default_volume_envelope, TRUE); + check_volume_envelope(synth, &download, &midi, &default_volume_envelope, FALSE);
/* there is a bank fallback on the drum channel */ download = default_instrument_download; @@ -2019,7 +2019,7 @@ static void test_instrument_selection(void) midi.messages[2] = make_program_change(0, 9, 0); midi.messages[3] = make_note_on(0, 9, 60, 127); midi.messages[4] = make_note_off(10000000, 9, 60, 127); - check_volume_envelope(synth, &download, &midi, &default_volume_envelope, TRUE); + check_volume_envelope(synth, &download, &midi, &default_volume_envelope, FALSE);
IDirectMusicSynth_Release(synth); }