From 93e99ba30d97e141bedd1b28aa7aac97faa23c0f Mon Sep 17 00:00:00 2001
From: Filippos Karapetis
Date: Sat, 27 Feb 2016 14:40:13 +0200
Subject: SCI: Remove a TODO in GK1 that has been resolved properly now

---
 engines/sci/graphics/screen_item32.cpp | 17 -----------------
 1 file changed, 17 deletions(-)

(limited to 'engines/sci')

diff --git a/engines/sci/graphics/screen_item32.cpp b/engines/sci/graphics/screen_item32.cpp
index fe995a429a..80d03086ae 100644
--- a/engines/sci/graphics/screen_item32.cpp
+++ b/engines/sci/graphics/screen_item32.cpp
@@ -202,23 +202,6 @@ void ScreenItem::setFromObject(SegManager *segMan, const reg_t object, const boo
 		writeSelectorValue(segMan, object, SELECTOR(priority), _position.y);
 	}
 
-	// TODO: At this point (needs checking), GK1 seems to check for the "visible"
-	// selector of a plane object. If the object has such a selector, and it's set
-	// to 0, then the object should be hidden.
-	//
-	// This is needed for the inventory in GK1, and seemed to be used only for that
-	// game - the "visible" selector isn't present in any other SCI32 game.
-	// Possible disabled and unverified code that checks for this follows. This fixes
-	// the inventory in GK1. Verify against disassembly!
-#if 0
-	if (lookupSelector(segMan, object, SELECTOR(visible), NULL, NULL) != kSelectorNone) {
-		if (readSelectorValue(segMan, object, SELECTOR(visible)) == 0) {
-			_fixPriority = true;
-			_priority = -1;
-		}
-	}
-#endif
-
 	_z = readSelectorValue(segMan, object, SELECTOR(z));
 	_position.y -= _z;
 
-- 
cgit v1.2.3