From 56e59aaea60857dc5948641bd55e433e85fdad71 Mon Sep 17 00:00:00 2001
From: Johannes Schickel
Date: Fri, 21 Sep 2012 23:54:42 +0200
Subject: AGOS: Rewrite note on events with velocity 0 to note off events in
 MidiParser_S1D.

---
 engines/agos/midiparser_s1d.cpp | 9 +++++++++
 1 file changed, 9 insertions(+)

(limited to 'engines')

diff --git a/engines/agos/midiparser_s1d.cpp b/engines/agos/midiparser_s1d.cpp
index 54f0e9e03b..bef7199a98 100644
--- a/engines/agos/midiparser_s1d.cpp
+++ b/engines/agos/midiparser_s1d.cpp
@@ -101,6 +101,15 @@ void MidiParser_S1D::parseNextEvent(EventInfo &info) {
 		case 0x9: // note on
 			info.basic.param1 = *_position._playPos++;
 			info.basic.param2 = *_position._playPos++;
+			// Rewrite note on events with velocity 0 as note off events.
+			// This is the actual meaning of this, but theoretically this
+			// should not need to be rewritten, since all MIDI devices should
+			// interpret it like that. On the other hand all our MidiParser
+			// implementations do it and there seems to be code in MidiParser
+			// which relies on this for tracking active notes.
+			if (info.basic.param2 == 0) {
+				info.event = info.channel() | 0x80;
+			}
 			break;
 
 		case 0xA: { // loop control
-- 
cgit v1.2.3