From 948e4487389f6ae99d4a61e0cf6d995d16e62c82 Mon Sep 17 00:00:00 2001
From: Colin Snover
Date: Fri, 1 Jul 2016 15:54:52 -0500
Subject: SCI32: Fix signed comparison warnings

---
 engines/sci/graphics/frameout.cpp | 8 ++++----
 engines/sci/graphics/plane32.h    | 2 --
 2 files changed, 4 insertions(+), 6 deletions(-)

(limited to 'engines/sci/graphics')

diff --git a/engines/sci/graphics/frameout.cpp b/engines/sci/graphics/frameout.cpp
index fd37020896..140c27a6d7 100644
--- a/engines/sci/graphics/frameout.cpp
+++ b/engines/sci/graphics/frameout.cpp
@@ -696,7 +696,7 @@ void GfxFrameout::calcLists(ScreenItemListList &drawLists, EraseListList &eraseL
 	}
 
 	PlaneList::size_type planeCount = _planes.size();
-	for (int outerPlaneIndex = 0; outerPlaneIndex < planeCount; ++outerPlaneIndex) {
+	for (PlaneList::size_type outerPlaneIndex = 0; outerPlaneIndex < planeCount; ++outerPlaneIndex) {
 		const Plane *outerPlane = _planes[outerPlaneIndex];
 		const Plane *visiblePlane = _visiblePlanes.findByObject(outerPlane->_object);
 
@@ -742,7 +742,7 @@ void GfxFrameout::calcLists(ScreenItemListList &drawLists, EraseListList &eraseL
 		}
 
 		if (addedToEraseList) {
-			for (int rectIndex = 0; rectIndex < eraseList.size(); ++rectIndex) {
+			for (RectList::size_type rectIndex = 0; rectIndex < eraseList.size(); ++rectIndex) {
 				const Common::Rect &rect = *eraseList[rectIndex];
 				for (int innerPlaneIndex = planeCount - 1; innerPlaneIndex >= 0; --innerPlaneIndex) {
 					const Plane &innerPlane = *_planes[innerPlaneIndex];
@@ -813,7 +813,7 @@ void GfxFrameout::calcLists(ScreenItemListList &drawLists, EraseListList &eraseL
 
 			eraseList.add(outerPlane._screenRect.findIntersectingRect(visibleOuterPlane->_screenRect));
 
-			for (PlaneList::size_type innerIndex = planeCount - 1; innerIndex >= 0; --innerIndex) {
+			for (int innerIndex = (int)planeCount - 1; innerIndex >= 0; --innerIndex) {
 				// "inner" just refers to the inner loop
 				const Plane &innerPlane = *_planes[innerIndex];
 				const Plane *visibleInnerPlane = _visiblePlanes.findByObject(innerPlane._object);
@@ -903,7 +903,7 @@ void GfxFrameout::calcLists(ScreenItemListList &drawLists, EraseListList &eraseL
 			}
 
 			if (_planes[planeIndex]->_type == kPlaneTypeTransparent) {
-				for (PlaneList::size_type i = planeIndex - 1; i >= 0; --i) {
+				for (int i = (int)planeIndex - 1; i >= 0; --i) {
 					_planes[i]->filterDownEraseRects(drawLists[i], eraseLists[i], eraseLists[planeIndex]);
 				}
 
diff --git a/engines/sci/graphics/plane32.h b/engines/sci/graphics/plane32.h
index acd535e75a..a1739264e8 100644
--- a/engines/sci/graphics/plane32.h
+++ b/engines/sci/graphics/plane32.h
@@ -495,8 +495,6 @@ private:
 	using PlaneListBase::push_back;
 
 public:
-	typedef int size_type;
-
 	// A method for finding the index of a plane inside a
 	// PlaneList is used because entries in the main plane
 	// list and visible plane list of GfxFrameout are
-- 
cgit v1.2.3