From d2871f925af38da89362b45edac43dbed5c925f5 Mon Sep 17 00:00:00 2001
From: Max Horn
Date: Fri, 5 Feb 2010 22:55:41 +0000
Subject: SCI: Add comment to GfxCompare::canBeHereCheckRectList

svn-id: r47919
---
 engines/sci/graphics/compare.cpp | 5 +++++
 1 file changed, 5 insertions(+)

(limited to 'engines')

diff --git a/engines/sci/graphics/compare.cpp b/engines/sci/graphics/compare.cpp
index f3d7e4b6cc..697324f975 100644
--- a/engines/sci/graphics/compare.cpp
+++ b/engines/sci/graphics/compare.cpp
@@ -93,6 +93,11 @@ bool GfxCompare::canBeHereCheckRectList(reg_t checkObject, Common::Rect checkRec
 				curRect.right = GET_SEL32V(_segMan, curObject, SELECTOR(brRight));
 				curRect.bottom = GET_SEL32V(_segMan, curObject, SELECTOR(brBottom));
 				// Check if curRect is within checkRect
+				// TODO: This check is slightly odd, because it means that a rect is not contained
+				// in itself. It may very well be that the original SCI engine did it just
+				// this way, so it should not be changed lightly. However, somebody should
+				// confirm whether the original engine really did it this way. Then, update
+				// this comment accordingly, and, if necessary, fix the code.
 				if (curRect.right > checkRect.left &&
 				    curRect.left < checkRect.right &&
 				    curRect.bottom > checkRect.top &&
-- 
cgit v1.2.3