From 29efb5d6ba1e0e342a29a89cb53a1432c16bb09c Mon Sep 17 00:00:00 2001 From: Johannes Lorenz <1042576+JohannesLorenz@users.noreply.github.com> Date: Sat, 2 Jul 2022 10:20:46 +0200 Subject: [PATCH] MidiRawClient: Fix setting useless param (#6431) MidiChannelPressure and MidiProgramChange have only one param, so only set this one param for those. --- src/core/midi/MidiClient.cpp | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/src/core/midi/MidiClient.cpp b/src/core/midi/MidiClient.cpp index af7cfee0a..1c200bc1b 100644 --- a/src/core/midi/MidiClient.cpp +++ b/src/core/midi/MidiClient.cpp @@ -203,12 +203,15 @@ void MidiClientRaw::parseData( const unsigned char c ) case MidiNoteOff: case MidiNoteOn: case MidiKeyPressure: - case MidiChannelPressure: - case MidiProgramChange: m_midiParseData.m_midiEvent.setKey(m_midiParseData.m_buffer[0]); m_midiParseData.m_midiEvent.setVelocity(m_midiParseData.m_buffer[1]); break; + case MidiChannelPressure: + case MidiProgramChange: + m_midiParseData.m_midiEvent.setParam(0, m_midiParseData.m_buffer[0]); + break; + case MidiControlChange: m_midiParseData.m_midiEvent.setControllerNumber(m_midiParseData.m_buffer[0]); m_midiParseData.m_midiEvent.setControllerValue( m_midiParseData.m_buffer[1]);