cpp/openttd-patchpack/source Changeset - r14740:9c3af6ca7b4c

Changeset - r14740:9c3af6ca7b4c

Parent rev.

Child rev.

[Not reviewed]

master

0 6 0

terkhen - 14 years ago 2010-03-06 12:42:53
terkhen@openttd.org

(svn r19338) -Codechange: Move the acceleration cache to GroundVehicle.

6 files changed with 56 insertions and 55 deletions:

src/ground_vehicle.hpp

src/newgrf_engine.cpp

src/train.h

src/train_cmd.cpp

src/vehicle_cmd.cpp

src/vehicle_gui.cpp

0 comments (0 inline, 0 general)

src/ground_vehicle.hpp

➞

Show inline comments

@@ @@ -14,11 +14,35 @@ @@
 #include "vehicle_base.h"
 /** What is the status of our acceleration? */
 enum AccelStatus {
 	AS_ACCEL, ///< We want to go faster, if possible of course.
 	AS_BRAKE  ///< We want to stop.
 };
 /**
  * Cached acceleration values.
  * All of these values except cached_slope_resistance are set only for the first part of a vehicle.
  */
 struct AccelerationCache {
 	/* Cached values, recalculated when the cargo on a vehicle changes (in addition to the conditions below) */
 	uint32 cached_weight;           ///< Total weight of the consist.
 	uint32 cached_slope_resistance; ///< Resistance caused by weight when this vehicle part is at a slope.
 	uint32 cached_max_te;           ///< Maximum tractive effort of consist.
 	/* Cached values, recalculated on load and each time a vehicle is added to/removed from the consist. */
 	uint32 cached_power;            ///< Total power of the consist.
 	uint32 cached_air_drag;         ///< Air drag coefficient of the vehicle.
 	uint16 cached_axle_resistance;  ///< Resistance caused by the axles of the vehicle.
 	uint16 cached_max_track_speed;  ///< Maximum consist speed limited by track type.
 };
 /**
  * Base class for all vehicles that move through ground.
  */
 template <class T, VehicleType Type>
 struct GroundVehicle : public SpecializedVehicle<T, Type> {
 	AccelerationCache acc_cache;
 	/**
 	 * The constructor at SpecializedVehicle must be called.

src/newgrf_engine.cpp

➞

Show inline comments

@@ @@ -778,10 +778,10 @@ static uint32 VehicleGetVariable(const R @@
 				case 0x62: return t->track;
 				case 0x66: return t->railtype;
 				case 0x73: return t->tcache.cached_veh_length;
 				case 0x74: return t->tcache.cached_power;
 				case 0x75: return GB(t->tcache.cached_power,  8, 24);
 				case 0x76: return GB(t->tcache.cached_power, 16, 16);
 				case 0x77: return GB(t->tcache.cached_power, 24,  8);
 				case 0x74: return t->acc_cache.cached_power;
 				case 0x75: return GB(t->acc_cache.cached_power,  8, 24);
 				case 0x76: return GB(t->acc_cache.cached_power, 16, 16);
 				case 0x77: return GB(t->acc_cache.cached_power, 24,  8);
 				case 0x7C: return t->First()->index;
 				case 0x7D: return GB(t->First()->index, 8, 8);
 				case 0x7F: return 0; // Used for vehicle reversing hack in TTDP

src/train.h

➞

Show inline comments

@@ @@ -56,25 +56,8 @@ bool TryPathReserve(Train *v, bool mark_ @@
 int GetTrainStopLocation(StationID station_id, TileIndex tile, const Train *v, int *station_ahead, int *station_length);
 /**
  * Cached acceleration values.
  * All of these values except cached_slope_resistance are set only for the first part of a vehicle.
  */
 struct AccelerationCache {
 	/* cached values, recalculated when the cargo on a train changes (in addition to the conditions above) */
 	uint32 cached_weight;           ///< total weight of the consist.
 	uint32 cached_slope_resistance; ///< Resistance caused by weight when this vehicle part is at a slope
 	uint32 cached_max_te;           ///< max tractive effort of consist
 	/* cached values, recalculated on load and each time a vehicle is added to/removed from the consist. */
 	uint32 cached_power;            ///< total power of the consist.
 	uint32 cached_air_drag;         ///< Air drag coefficient of the vehicle
 	uint16 cached_axle_resistance;  ///< Resistance caused by the axles of the vehicle
 	uint16 cached_max_track_speed;  ///< Max consist speed limited by track type
 };
 /** Variables that are cached to improve performance and such */
-struct TrainCache : public AccelerationCache {
 struct TrainCache {
 	/* Cached wagon override spritegroup */
 	const struct SpriteGroup *cached_override;
@@ @@ -102,12 +85,6 @@ struct TrainCache : public AccelerationC @@
 	EngineID first_engine;  ///< cached EngineID of the front vehicle. INVALID_ENGINE for the front vehicle itself.
 };
 /** What is the status of our acceleration? */
 enum AccelStatus {
 	AS_ACCEL, ///< We want to go faster, if possible ofcourse
 	AS_BRAKE  ///< We want to stop
 };
 /**
  * 'Train' is either a loco or a wagon.
  */
@@ @@ -486,9 +463,9 @@ protected: // These functions should not @@
 		for (const Train *u = this; u != NULL; u = u->Next()) {
 			if (HasBit(u->flags, VRF_GOINGUP)) {
-				incl += u->tcache.cached_slope_resistance;
+				incl += u->acc_cache.cached_slope_resistance;
 			} else if (HasBit(u->flags, VRF_GOINGDOWN)) {
-				incl -= u->tcache.cached_slope_resistance;
+				incl -= u->acc_cache.cached_slope_resistance;
+			}
+		}

src/train_cmd.cpp

➞

Show inline comments

@@ @@ -115,22 +115,22 @@ void Train::PowerChanged() @@
 		if (track_speed > 0) max_track_speed = min(max_track_speed, track_speed);
+	}
 	this->tcache.cached_axle_resistance = 60 * number_of_parts;
 	this->tcache.cached_air_drag = 20 + 3 * number_of_parts;
 	this->acc_cache.cached_axle_resistance = 60 * number_of_parts;
 	this->acc_cache.cached_air_drag = 20 + 3 * number_of_parts;
 	max_te *= 10000; // Tractive effort in (tonnes * 1000 * 10 =) N
 	max_te /= 256;   // Tractive effort is a [0-255] coefficient
-	if (this->tcache.cached_power != total_power || this->tcache.cached_max_te != max_te) {
+	if (this->acc_cache.cached_power != total_power || this->acc_cache.cached_max_te != max_te) {
 		/* Stop the vehicle if it has no power */
 		if (total_power == 0) this->vehstatus |= VS_STOPPED;
 		this->tcache.cached_power = total_power;
 		this->tcache.cached_max_te = max_te;
 		this->acc_cache.cached_power = total_power;
 		this->acc_cache.cached_max_te = max_te;
 		SetWindowDirty(WC_VEHICLE_DETAILS, this->index);
 		SetWindowWidgetDirty(WC_VEHICLE_VIEW, this->index, VVW_WIDGET_START_STOP_VEH);
+	}
-	this->tcache.cached_max_track_speed = max_track_speed;
+	this->acc_cache.cached_max_track_speed = max_track_speed;
+}
@@ @@ -146,11 +146,11 @@ void Train::CargoChanged() @@
 	for (Train *u = this; u != NULL; u = u->Next()) {
 		uint32 current_weight = u->GetWeight();
 		weight += current_weight;
-		u->tcache.cached_slope_resistance = current_weight * u->GetSlopeSteepness();
+		u->acc_cache.cached_slope_resistance = current_weight * u->GetSlopeSteepness();
+	}
 	/* store consist weight in cache */
-	this->tcache.cached_weight = weight;
+	this->acc_cache.cached_weight = weight;
 	/* Now update train power (tractive effort is dependent on weight) */
 	this->PowerChanged();
@@ @@ -498,7 +498,7 @@ int Train::GetCurrentMaxSpeed() const @@
+		}
+	}
-	return min(max_speed, this->tcache.cached_max_track_speed);
+	return min(max_speed, this->acc_cache.cached_max_track_speed);
+}
 /**
@@ @@ -511,10 +511,10 @@ int Train::GetAcceleration() const @@
 	int32 speed = this->GetCurrentSpeed();
 	/* Weight is stored in tonnes */
-	int32 mass = this->tcache.cached_weight;
+	int32 mass = this->acc_cache.cached_weight;
 	/* Power is stored in HP, we need it in watts. */
-	int32 power = this->tcache.cached_power * 746;
+	int32 power = this->acc_cache.cached_power * 746;
 	int32 resistance = 0;
@@ @@ -523,11 +523,11 @@ int Train::GetAcceleration() const @@
 	const int area = 120;
 	if (!maglev) {
 		resistance = (13 * mass) / 10;
-		resistance += this->tcache.cached_axle_resistance;
+		resistance += this->acc_cache.cached_axle_resistance;
 		resistance += (this->GetRollingFriction() * mass * speed) / 1000;
-		resistance += (area * this->tcache.cached_air_drag * speed * speed) / 10000;
+		resistance += (area * this->acc_cache.cached_air_drag * speed * speed) / 10000;
 	} else {
-		resistance += (area * this->tcache.cached_air_drag * speed * speed) / 20000;
+		resistance += (area * this->acc_cache.cached_air_drag * speed * speed) / 20000;
+	}
 	resistance += this->GetSlopeResistance();
@@ @@ -536,7 +536,7 @@ int Train::GetAcceleration() const @@
 	/* This value allows to know if the vehicle is accelerating or braking. */
 	AccelStatus mode = this->GetAccelerationStatus();
-	const int max_te = this->tcache.cached_max_te; // [N]
+	const int max_te = this->acc_cache.cached_max_te; // [N]
 	int force;
 	if (speed > 0) {
 		if (!maglev) {
@@ @@ -564,10 +564,10 @@ void Train::UpdateAcceleration() @@
+{
 	assert(this->IsFrontEngine());
 	this->max_speed = this->tcache.cached_max_track_speed;
 	uint power = this->tcache.cached_power;
 	uint weight = this->tcache.cached_weight;
 	this->max_speed = this->acc_cache.cached_max_track_speed;
 	uint power = this->acc_cache.cached_power;
 	uint weight = this->acc_cache.cached_weight;
 	assert(weight != 0);
 	this->acceleration = Clamp(power / weight * 4, 1, 255);
+}
@@ @@ -2265,7 +2265,7 @@ static bool CheckTrainStayInDepot(Train @@
+	}
 	/* if the train got no power, then keep it in the depot */
-	if (v->tcache.cached_power == 0) {
+	if (v->acc_cache.cached_power == 0) {
 		v->vehstatus |= VS_STOPPED;
 		SetWindowDirty(WC_VEHICLE_DEPOT, v->tile);
 		return true;

src/vehicle_cmd.cpp

➞

Show inline comments

@@ @@ -78,7 +78,7 @@ CommandCost CmdStartStopVehicle(TileInde @@
 	switch (v->type) {
 		case VEH_TRAIN:
-			if ((v->vehstatus & VS_STOPPED) && Train::From(v)->tcache.cached_power == 0) return_cmd_error(STR_ERROR_TRAIN_START_NO_CATENARY);
+			if ((v->vehstatus & VS_STOPPED) && Train::From(v)->acc_cache.cached_power == 0) return_cmd_error(STR_ERROR_TRAIN_START_NO_CATENARY);
 			break;
 		case VEH_SHIP:

src/vehicle_gui.cpp

➞

Show inline comments

@@ @@ -1525,9 +1525,9 @@ struct VehicleDetailsWindow : Window { @@
 				switch (v->type) {
 					case VEH_TRAIN:
 						SetDParam(2, v->GetDisplayMaxSpeed());
 						SetDParam(1, Train::From(v)->tcache.cached_power);
 						SetDParam(0, Train::From(v)->tcache.cached_weight);
 						SetDParam(3, Train::From(v)->tcache.cached_max_te / 1000);
 						SetDParam(1, Train::From(v)->acc_cache.cached_power);
 						SetDParam(0, Train::From(v)->acc_cache.cached_weight);
 						SetDParam(3, Train::From(v)->acc_cache.cached_max_te / 1000);
 						DrawString(r.left + WD_FRAMERECT_LEFT, r.right - WD_FRAMERECT_RIGHT, y, (_settings_game.vehicle.train_acceleration_model != AM_ORIGINAL && GetRailTypeInfo(Train::From(v)->railtype)->acceleration_type != 2) ?
 								STR_VEHICLE_INFO_WEIGHT_POWER_MAX_SPEED_MAX_TE : STR_VEHICLE_INFO_WEIGHT_POWER_MAX_SPEED);
 						break;
@@ @@ -1963,7 +1963,7 @@ public: @@
 		} else if (v->vehstatus & VS_STOPPED) {
 			if (v->type == VEH_TRAIN) {
 				if (v->cur_speed == 0) {
-					if (Train::From(v)->tcache.cached_power == 0) {
+					if (Train::From(v)->acc_cache.cached_power == 0) {
 						str = STR_VEHICLE_STATUS_TRAIN_NO_POWER;
 					} else {
 						str = STR_VEHICLE_STATUS_STOPPED;

0 comments (0 inline, 0 general)