MidiThru: Can now block midi spam

MIDIThruBlockSpam=1
When enabled we dont forward 1 byte messages of MIDI_TIMING_CLOCK or MIDI_ACTIVE_SENSING
if defaults to off => keep forwarding these

Author: penfold42

src/config.cpp

@@ -123,6 +123,8 @@ void CConfig::Load (void)
 		}
 	}
 
+	m_bMIDIThruBlockSpam = m_Properties.GetNumber ("MIDIThruBlockSpam", 0) != 0;
+
 	m_bMIDIRXProgramChange = m_Properties.GetNumber ("MIDIRXProgramChange", 1) != 0;
 	m_bIgnoreAllNotesOff = m_Properties.GetNumber ("IgnoreAllNotesOff", 0) != 0;
 	m_bMIDIAutoVoiceDumpOnPC = m_Properties.GetNumber ("MIDIAutoVoiceDumpOnPC", 0) != 0;
@@ -361,6 +363,11 @@ const char *CConfig::GetMIDIThru2Out (void) const
 	return m_MIDIThru2Out.c_str ();
 }
 
+bool CConfig::GetMIDIThruBlockSpam (void) const
+{
+	return m_bMIDIThruBlockSpam;
+}
+
 bool CConfig::GetMIDIRXProgramChange (void) const
 {
 	return m_bMIDIRXProgramChange;

src/config.h

@@ -126,6 +126,7 @@ class CConfig		// Configuration for MiniDexed
 	const char *GetMIDIThruOut (void) const;	// "" if not specified
 	const char *GetMIDIThru2In (void) const;	// "" if not specified
 	const char *GetMIDIThru2Out (void) const;	// "" if not specified
+	bool GetMIDIThruBlockSpam (void) const;		// false if not specified
 	bool GetMIDIRXProgramChange (void) const;	// true if not specified
 	bool GetIgnoreAllNotesOff (void) const;
 	bool GetMIDIAutoVoiceDumpOnPC (void) const; // false if not specified
@@ -282,6 +283,7 @@ class CConfig		// Configuration for MiniDexed
 	std::string m_MIDIThruOut;
 	std::string m_MIDIThru2In;
 	std::string m_MIDIThru2Out;
+	bool m_bMIDIThruBlockSpam;
 	bool m_bMIDIRXProgramChange;
 	bool m_bIgnoreAllNotesOff;
 	bool m_bMIDIAutoVoiceDumpOnPC;

src/mididevice.cpp

@@ -189,27 +189,34 @@ void CMIDIDevice::MIDIMessageHandler (const u8 *pMessage, size_t nLength, unsign
 	}
 */
 
+	bool blockSpamThru = m_pConfig->GetMIDIThruBlockSpam();
 	// Handle MIDI Thru
-	if (m_DeviceName.compare (m_pConfig->GetMIDIThruIn ()) == 0)
-	{
-		TDeviceMap::const_iterator Iterator;
+	bool canThru = ( (nLength > 1) || !blockSpamThru
+		|| (pMessage[0] != MIDI_TIMING_CLOCK && pMessage[0] != MIDI_ACTIVE_SENSING) );
 
-		Iterator = s_DeviceMap.find (m_pConfig->GetMIDIThruOut ());
-		if (Iterator != s_DeviceMap.end ())
+	if (canThru)
+	{
+		if (m_DeviceName.compare (m_pConfig->GetMIDIThruIn ()) == 0)
 		{
-			Iterator->second->Send (pMessage, nLength, nCable);
-		}
-	}
+			TDeviceMap::const_iterator Iterator;
 
-	// Handle MIDI Thru 2
-	if (m_DeviceName.compare (m_pConfig->GetMIDIThru2In ()) == 0)
-	{
-		TDeviceMap::const_iterator Iterator;
+			Iterator = s_DeviceMap.find (m_pConfig->GetMIDIThruOut ());
+			if (Iterator != s_DeviceMap.end ())
+			{
+				Iterator->second->Send (pMessage, nLength, nCable);
+			}
+		}
 
-		Iterator = s_DeviceMap.find (m_pConfig->GetMIDIThru2Out ());
-		if (Iterator != s_DeviceMap.end ())
+		// Handle MIDI Thru 2
+		if (m_DeviceName.compare (m_pConfig->GetMIDIThru2In ()) == 0)
 		{
-			Iterator->second->Send (pMessage, nLength, nCable);
+			TDeviceMap::const_iterator Iterator;
+
+			Iterator = s_DeviceMap.find (m_pConfig->GetMIDIThru2Out ());
+			if (Iterator != s_DeviceMap.end ())
+			{
+				Iterator->second->Send (pMessage, nLength, nCable);
+			}
 		}
 	}