From 7731ded5afe4b246c8fef38b4a70602303c6dff6 Mon Sep 17 00:00:00 2001
From: Matthew Hoops
Date: Sun, 7 Feb 2010 17:18:14 +0000
Subject: Update comments in autoDetectSci21KernelType() explaining what we're
 checking for.

svn-id: r47973
---
 engines/sci/engine/features.cpp | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/engines/sci/engine/features.cpp b/engines/sci/engine/features.cpp
index 57bf12c391..1aaaa39fa8 100644
--- a/engines/sci/engine/features.cpp
+++ b/engines/sci/engine/features.cpp
@@ -394,8 +394,10 @@ bool GameFeatures::autoDetectSci21KernelType() {
 		if (opcode == op_callk) {
 			uint16 kFuncNum = opparams[0];
 
-			// TODO: Explain this check; what are those kernel funcs supposed
-			//  to be, why does this check work like it does?
+			// Here we check for the kDoSound opcode that's used in SCI2.1.
+			// Finding 0x40 as kDoSound in the Sound::play() function means the game is using
+			// the modified SCI2 kernel table found in some older SCI2.1 games (GK2 demo, KQ7 v1.4).
+			// Finding 0x75 as kDoSound means the game is using the regular SCI2.1 kernel table.
 			if (kFuncNum == 0x40) {
 				_sci21KernelType = SCI_VERSION_2;
 				return true;
-- 
cgit v1.2.3