[http://sourceforge.net/p/ufoai/feature-requests/292 Item 292] imported from sourceforge.net tracker on 2013-01-28 20:16:10

Ok, check out the following Grid:

A B
C D

"A","B" & "C" and soldiers and "D" is an empty space. Currently in 2.2 a soldier "A" can't move to "D" even though there is plenty of space between "C" & "D" for him to pass. This can make it awkward maneuvering your troops in narrow spaces.

It feels really arbitrary, though i'm sure it was simplest to code it that way.
===== Comments Ported from Sourceforge =====

====== tlh2000 (2008-01-30 07:58:53) ======

Logged In: YES
user_id=116930
Originator: NO

maybe some special check in the forbidden list check can "fix" this?
====== hoehrer (2008-01-30 09:52:21) ======

Logged In: YES
user_id=1434318
Originator: NO

The patch below _might_ fix this (may look strange for very big soldiers - i.e. future battlesuits; Also, this will also make it work for _any_ moving entity in battlescape)

But this patch/change has to be tested very carefully.

Werner


=======================
Index: src/common/cmodel.c
===================================================================
--- src/common/cmodel.c (Revision 14333)
+++ src/common/cmodel.c (Arbeitskopie)
@@ -2563,9 +2563,7 @@
/* If direction vector index is set to a diagonal offset check if we can move there through connected "straight" squares. */
if (dir > 3 &&
!( (dx > 0 ? R_CONN_PX(map, x, y+dy, z) : R_CONN_NX(map, x, y+dy, z))
- && (dy > 0 ? R_CONN_PY(map, x+dx, y, z) : R_CONN_NY(map, x+dx, y, z))
- && !Grid_CheckForbidden(map, x, y+dy, z)
- && !Grid_CheckForbidden(map, x+dx, y, z)))
+ && (dy > 0 ? R_CONN_PY(map, x+dx, y, z) : R_CONN_NY(map, x+dx, y, z)))
return;
break;
case ACTOR_SIZE_2x2:
=======================
====== geever (2009-01-14 12:08:01) ======

With the new pathfinding code this is possible.

-geever