summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorrubidium <rubidium@openttd.org>2007-09-09 11:23:49 +0000
committerrubidium <rubidium@openttd.org>2007-09-09 11:23:49 +0000
commit40b871410b557aa8fdf6c2ea1d872a2563c922ab (patch)
treef20482e13f3272992a0ec242e6105684bc4d19b0 /src
parentefea7622acb6dbf5b4a75be3e0b0c6d016e5feb1 (diff)
downloadopenttd-40b871410b557aa8fdf6c2ea1d872a2563c922ab.tar.xz
(svn r11068) -Codechange: remove Vehicle::HasFront as all vehicles have the Vehicle::first pointer correctly set.
Diffstat (limited to 'src')
-rw-r--r--src/depot_gui.cpp2
-rw-r--r--src/group_cmd.cpp2
-rw-r--r--src/newgrf_engine.cpp4
-rw-r--r--src/order_gui.cpp2
-rw-r--r--src/roadveh.h1
-rw-r--r--src/train.h1
-rw-r--r--src/tunnelbridge_cmd.cpp2
-rw-r--r--src/vehicle.h6
8 files changed, 5 insertions, 15 deletions
diff --git a/src/depot_gui.cpp b/src/depot_gui.cpp
index 318ee3f3d..9c0e87da1 100644
--- a/src/depot_gui.cpp
+++ b/src/depot_gui.cpp
@@ -474,7 +474,7 @@ static void HandleCloneVehClick(const Vehicle *v, const Window *w)
if (v == NULL) return;
- if (v->HasFront() && !v->IsPrimaryVehicle()) {
+ if (!v->IsPrimaryVehicle()) {
v = v->First();
/* Do nothing when clicking on a train in depot with no loc attached */
if (v->type == VEH_TRAIN && !IsFrontEngine(v)) return;
diff --git a/src/group_cmd.cpp b/src/group_cmd.cpp
index 7942d406e..2726c59f9 100644
--- a/src/group_cmd.cpp
+++ b/src/group_cmd.cpp
@@ -354,7 +354,7 @@ CommandCost CmdSetGroupReplaceProtection(TileIndex tile, uint32 flags, uint32 p1
*/
void RemoveVehicleFromGroup(const Vehicle *v)
{
- if (!v->IsValid() || !(v->HasFront() && v->IsPrimaryVehicle())) return;
+ if (!v->IsValid() || !v->IsPrimaryVehicle()) return;
if (!IsDefaultGroupID(v->group_id)) DecreaseGroupNumVehicle(v->group_id);
}
diff --git a/src/newgrf_engine.cpp b/src/newgrf_engine.cpp
index 9272a6d52..76e4982db 100644
--- a/src/newgrf_engine.cpp
+++ b/src/newgrf_engine.cpp
@@ -509,8 +509,6 @@ static uint32 VehicleGetVariable(const ResolverObject *object, byte variable, by
switch (variable) {
case 0x40: // Get length of consist
case 0x41: // Get length of same consecutive wagons
- if (!v->HasFront()) return 1;
-
{
const Vehicle* u;
byte chain_before = 0;
@@ -832,7 +830,7 @@ static inline void NewVehicleResolver(ResolverObject *res, EngineID engine_type,
res->ResolveReal = &VehicleResolveReal;
res->u.vehicle.self = v;
- res->u.vehicle.parent = (v != NULL && v->HasFront()) ? v->First() : v;
+ res->u.vehicle.parent = (v != NULL) ? v->First() : v;
res->u.vehicle.self_type = engine_type;
diff --git a/src/order_gui.cpp b/src/order_gui.cpp
index 0bf98f240..2dd5714a8 100644
--- a/src/order_gui.cpp
+++ b/src/order_gui.cpp
@@ -355,7 +355,7 @@ static bool HandleOrderVehClick(const Vehicle *v, const Vehicle *u, Window *w)
{
if (u->type != v->type) return false;
- if (u->HasFront() && !u->IsPrimaryVehicle()) {
+ if (!u->IsPrimaryVehicle()) {
u = u->First();
if (!u->IsPrimaryVehicle()) return false;
}
diff --git a/src/roadveh.h b/src/roadveh.h
index 1c545ea43..46e14bac4 100644
--- a/src/roadveh.h
+++ b/src/roadveh.h
@@ -69,7 +69,6 @@ struct RoadVehicle : public Vehicle {
ExpensesType GetExpenseType(bool income) const { return income ? EXPENSES_ROADVEH_INC : EXPENSES_ROADVEH_RUN; }
WindowClass GetVehicleListWindowClass() const { return WC_ROADVEH_LIST; }
bool IsPrimaryVehicle() const { return IsRoadVehFront(this); }
- bool HasFront() const { return true; }
int GetImage(Direction direction) const;
int GetDisplaySpeed() const { return this->cur_speed * 10 / 32; }
int GetDisplayMaxSpeed() const { return this->max_speed * 10 / 32; }
diff --git a/src/train.h b/src/train.h
index 1751e07e7..51c6bf2a8 100644
--- a/src/train.h
+++ b/src/train.h
@@ -296,7 +296,6 @@ struct Train : public Vehicle {
WindowClass GetVehicleListWindowClass() const { return WC_TRAINS_LIST; }
void PlayLeaveStationSound() const;
bool IsPrimaryVehicle() const { return IsFrontEngine(this); }
- bool HasFront() const { return true; }
int GetImage(Direction direction) const;
int GetDisplaySpeed() const { return this->u.rail.last_speed * 10 / 16; }
int GetDisplayMaxSpeed() const { return this->u.rail.cached_max_speed * 10 / 16; }
diff --git a/src/tunnelbridge_cmd.cpp b/src/tunnelbridge_cmd.cpp
index 33f28a540..87abd7470 100644
--- a/src/tunnelbridge_cmd.cpp
+++ b/src/tunnelbridge_cmd.cpp
@@ -1369,7 +1369,7 @@ static uint32 VehicleEnter_TunnelBridge(Vehicle *v, TileIndex tile, int x, int y
} else if (IsBridge(tile)) { // XXX is this necessary?
DiagDirection dir;
- if (v->HasFront() && v->IsPrimaryVehicle()) {
+ if (v->IsPrimaryVehicle()) {
/* modify speed of vehicle */
uint16 spd = _bridge[GetBridgeType(tile)].speed;
diff --git a/src/vehicle.h b/src/vehicle.h
index 509a87e4c..7b2b230fc 100644
--- a/src/vehicle.h
+++ b/src/vehicle.h
@@ -421,12 +421,6 @@ public:
virtual bool IsPrimaryVehicle() const { return false; }
/**
- * Whether this vehicle understands the concept of a front engine, so
- * basically, if GetFirstVehicleInChain() can be called for it.
- */
- virtual bool HasFront() const { return false; }
-
- /**
* Gets the sprite to show for the given direction
* @param direction the direction the vehicle is facing
* @return the sprite for the given vehicle in the given direction