From 173a022f91ffb888292ced1a8a4756db5eaca9b1 Mon Sep 17 00:00:00 2001
From: Willem Jan Palenstijn
Date: Sat, 6 Apr 2013 20:48:52 +0200
Subject: SCI: Clarify comment

---
 engines/sci/sound/soundcmd.cpp | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

(limited to 'engines/sci/sound/soundcmd.cpp')

diff --git a/engines/sci/sound/soundcmd.cpp b/engines/sci/sound/soundcmd.cpp
index fb4564d859..f3375e7bb2 100644
--- a/engines/sci/sound/soundcmd.cpp
+++ b/engines/sci/sound/soundcmd.cpp
@@ -402,9 +402,11 @@ reg_t SoundCommandParser::kDoSoundFade(int argc, reg_t *argv, reg_t acc) {
 			musicSlot->fadeStep = volume > musicSlot->fadeTo ? -5 : 5;
 		musicSlot->fadeTickerStep = argv[2].toUint16() * 16667 / _music->soundGetTempo();
 		musicSlot->fadeTicker = 0;
-		// TODO: We handle this as a bit field (i.e. containing values 0 and 1),
-		// but some games pass other values here as well (e.g. some KQ6 scripts
-		// pass 3 here)
+
+		// argv[4] is a boolean. Scripts sometimes pass strange values,
+		// but SSCI ignores these. (Verified in KQ6.)
+		// KQ6 room 460 even passes an object, but treating this as 'true'
+		// seems fine in that case.
 		musicSlot->stopAfterFading = (argc == 5) ? (argv[4].toUint16() != 0) : false;
 		break;
 
-- 
cgit v1.2.3