# HG changeset patch
# User rubidium <rubidium@openttd.org>
# Date 2011-02-06 16:45:27
# Node ID abf58ea3223662c2308d51e8226d5197fcab12e4
# Parent  880f9cdb15bdebcd9f3084cb5c58f0b8a1d93ff6

(svn r21997) -Fix [FS#4473]: when the difference between force and resistance is smaller than the mass(*4) there would be no acceleration anymore, even when at higher (or lower) speed the force and resistance balance out better

diff --git a/src/ground_vehicle.cpp b/src/ground_vehicle.cpp
--- a/src/ground_vehicle.cpp
+++ b/src/ground_vehicle.cpp
@@ -151,8 +151,16 @@ int GroundVehicle<T, Type>::GetAccelerat
 	}
 
 	if (mode == AS_ACCEL) {
-		/* Divide by 4 to compensate for the wacky game scale. */
-		return (force - resistance) / (mass * 4);
+		/* Easy way out when there is no acceleration. */
+		if (force == resistance) return 0;
+
+		/* When we accelerate, make sure we always keep doing that, even when
+		 * the excess force is more than the mass. Otherwise a vehicle going
+		 * down hill will never slow down enough, and a vehicle that came up
+		 * a hill will never speed up enough to (eventually) get back to the
+		 * same (maximum) speed. */
+		int accel = (force - resistance) / (mass * 4);
+		return force < resistance ? min(-1, accel) : max(1, accel);
 	} else {
 		return min(-force - resistance, -10000) / mass;
 	}