- fixed last commit and optimized wall lookup code a bit.

No need to continue if a matching wall has been found.

source/core/rendering/scene/hw_walls.cpp

@@ -72,30 +72,29 @@ static walltype* IsOnWall(tspritetype* tspr, int height, DVector2& outpos)
 			// In Wanton Destruction's airplane level there's such a sprite assigned to the wrong sector.
 			if (d.X == 0)
 			{
-				if (fabs(tspr->pos.X - wal.pos.X) < maxorthdist);
+				if (fabs(tspr->pos.X - wal.pos.X) < maxorthdist)
 				{
-					closest = &wal;
+					return &wal;
 				}
 			}
 			else if (d.Y == 0)
 			{
-				if (fabs(tspr->pos.Y - wal.pos.Y) < maxorthdist);
+				if (fabs(tspr->pos.Y - wal.pos.Y) < maxorthdist)
 				{
-					closest = &wal;
+					return &wal;
 				}
 			}
 			else
 			{
 				double wdist = SquareDistToWall(tspr->pos.X, tspr->pos.Y, &wal, &outpos);
 				if (wdist <= maxdistsq)
 				{
-					closest = &wal;
+					return &wal;
 				}
 			}
 		}
 	}
-	// todo: cache this in the sprite to avoid recalculation.
-	return closest;
+	return nullptr;
 }
 
 //==========================================================================