From afa254721d786397d579230463ccdb96dcb9bae5 Mon Sep 17 00:00:00 2001
From: Walter van Niftrik
Date: Fri, 8 May 2009 12:39:37 +0000
Subject: SCI: AvoidPath: Fixed a regression caused by previous commit, and
 added a workaround for ECO room 300.

svn-id: r40383
---
 engines/sci/engine/kpathing.cpp | 33 +++++++++++++++++++++------------
 1 file changed, 21 insertions(+), 12 deletions(-)

(limited to 'engines/sci/engine')

diff --git a/engines/sci/engine/kpathing.cpp b/engines/sci/engine/kpathing.cpp
index 8956e63852..351bdb89f6 100644
--- a/engines/sci/engine/kpathing.cpp
+++ b/engines/sci/engine/kpathing.cpp
@@ -927,8 +927,8 @@ static int near_point(const Common::Point &p, Polygon *polygon, Common::Point *r
 		FloatPoint new_point;
 		uint32 new_dist;
 
-		// Ignore edges on the screen border
-		if (edge_on_screen_border(p1, p2))
+		// Ignore edges on the screen border, except for contained access polygons
+		if ((polygon->type != POLY_CONTAINED_ACCESS) && (edge_on_screen_border(p1, p2)))
 			continue;
 
 		// Compute near point
@@ -1142,7 +1142,7 @@ static Common::Point *fixup_end_point(PathfindingState *s, const Common::Point &
 		case POLY_CONTAINED_ACCESS:
 		case POLY_BARRED_ACCESS:
 		case POLY_NEAREST_ACCESS:
-			if (cont == CONT_INSIDE) {
+			if (cont != CONT_OUTSIDE) {
 				if (s->_appendPoint != NULL) {
 					// We shouldn't get here twice
 					warning("AvoidPath: end point is contained in multiple polygons");
@@ -1230,15 +1230,24 @@ static Polygon *convert_polygon(EngineState *s, reg_t polygon) {
 
 	// WORKAROUND: broken polygon in LSL1VGA, room 350, after opening elevator
 	// Polygon has 17 points but size is set to 19
-	if (s->_gameName == "LSL1") {
+	if ((size == 19) && (s->_gameName == "LSL1")) {
 		// FIXME: implement function to get current room number
 		if ((KP_UINT(s->script_000->locals_block->locals[13]) == 350)
-		&& (size == 19) && (read_point(list, is_reg_t, 18) == Common::Point(108, 137))) {
-			debug(1, "Applying fix for broken polygon in LSL1, room 350");
+		&& (read_point(list, is_reg_t, 18) == Common::Point(108, 137))) {
+			debug(1, "Applying fix for broken polygon in LSL1VGA, room 350");
 			size = 17;
 		}
 	}
 
+	// WORKAROUND: self-intersecting polygon in ECO, room 300, remove last point
+	if ((size == 11) && (s->_gameName == "eco")) {
+		if ((KP_UINT(s->script_000->locals_block->locals[13]) == 300)
+		&& (read_point(list, is_reg_t, 10) == Common::Point(221, 0))) {
+			debug(1, "Applying fix for self-intersecting polygon in ECO, room 300");
+			size = 10;
+		}
+	}
+
 	for (i = 0; i < size; i++) {
 		Vertex *vertex = new Vertex(read_point(list, is_reg_t, i));
 		poly->vertices.insertHead(vertex);
@@ -1662,6 +1671,12 @@ reg_t kAvoidPath(EngineState *s, int funct_nr, int argc, reg_t *argv) {
 
 		p = convert_polygon_set(s, poly_list, start, end, opt);
 
+		if (p && intersecting_polygons(p)) {
+			sciprintf("[avoidpath] Error: input set contains (self-)intersecting polygons\n");
+			delete p;
+			p = NULL;
+		}
+
 		if (!p) {
 			byte *oref;
 			sciprintf("[avoidpath] Error: pathfinding failed for following input:\n");
@@ -1677,12 +1692,6 @@ reg_t kAvoidPath(EngineState *s, int funct_nr, int argc, reg_t *argv) {
 			return output;
 		}
 
-		if (intersecting_polygons(p)) {
-			sciprintf("[avoidpath] Error: input set contains (self-)intersecting polygons\n");
-			delete p;
-			p = NULL;
-		}
-
 		dijkstra(p);
 
 		output = output_path(p, s);
-- 
cgit v1.2.3