Change Details

I was looking at the long-range pathfinder, hoping to see where the inefficiencies lied. I've posted my thoughts on the forum, but there are at least two things I believe we should do ASAP, and this diff does: - Remove ACCEPT_DIAGONAL_GAPS stuff, since we do not and never will (barring a pathfinder rewrite which would make it irrelevant anyways). - Since MakeGoalReachable returns a Point goal, we can simplify/Optimise OnTheWay considerably.

I was looking at the long-range pathfinder, hoping to see where the inefficiencies lied. I've posted my thoughts on the forum, but there are at least two things I believe we should do ASAP, and this diff does: - Remove ACCEPT_DIAGONAL_GAPS stuff, since we do not and never will (barring a pathfinder rewrite which would make it irrelevant anyways). - Since MakeGoalReachable returns a Point goal, we can simplify/Optimise OnTheWay considerably. - Reuse the PASSABLE macro everywhere

I was looking at the long-range pathfinder, hoping to see where the inefficiencies lied. I've posted my thoughts on the forum, but there are at least two things I believe we should do ASAP, and this diff does: - Remove ACCEPT_DIAGONAL_GAPS stuff, since we do not and never will (barring a pathfinder rewrite which would make it irrelevant anyways). - Since MakeGoalReachable returns a Point goal, we can simplify/Optimise OnTheWay considerably. - Reuse the PASSABLE macro everywhere