Fixed serious bug in removeOverlayElement() that would cause it to fail. This

caused getObjectUnderCursor() select the wrong object because the object list
was no longer guaranteed to be sorted on priority ("mask").

In Future Wars, this made it difficult (impossible?) to pick up the tunic,
because the game would pick the bushes instead, even though the tunic had a
higher priority.

svn-id: r21825

1 files changed, 1 insertions, 1 deletions

diff --git a/engines/cine/object.cpp b/engines/cine/object.cpp
index 47fee00b25..80c4b1999c 100644
--- a/engines/cine/object.cpp
+++ b/engines/cine/object.cpp
@@ -105,7 +105,7 @@ int8 removeOverlayElement(uint16 objIdx, uint16 param) {
 
 	currentHeadPtr = tempHead->next;
 
-	while (currentHeadPtr && (objIdx == currentHeadPtr->objIdx || param == currentHeadPtr->type)) {
+	while (currentHeadPtr && (objIdx != currentHeadPtr->objIdx || param != currentHeadPtr->type)) {
 		tempHead = currentHeadPtr;
 		currentHeadPtr = tempHead->next;
 	}