cpp/openttd-patchpack/source Changeset - r13929:bc7dece1f96b

Changeset - r13929:bc7dece1f96b

Parent rev.

Child rev.

[Not reviewed]

master

0 2 0

frosch - 15 years ago 2009-12-12 22:15:14
frosch@openttd.org

(svn r18471) -Codechange/Fix: [NoAI] Deduplicate code betweeen AIVehicle::SkipToVehicleOrder and AIOrder::SkipToOrder. They are the same. Also ORDER_CURRENT was not allowed for the latter, but well...

2 files changed with 3 insertions and 5 deletions:

src/ai/api/ai_order.cpp

src/ai/api/ai_vehicle.cpp

0 comments (0 inline, 0 general)

src/ai/api/ai_order.cpp

➞

Show inline comments

@@ @@ -388,24 +388,26 @@ static const Order *ResolveOrder(Vehicle @@
 /* static */ bool AIOrder::RemoveOrder(VehicleID vehicle_id, OrderPosition order_position)
+{
 	order_position = AIOrder::ResolveOrderPosition(vehicle_id, order_position);
 	EnforcePrecondition(false, IsValidVehicleOrder(vehicle_id, order_position));
 	return AIObject::DoCommand(0, vehicle_id, order_position, CMD_DELETE_ORDER);
+}
 /* static */ bool AIOrder::SkipToOrder(VehicleID vehicle_id, OrderPosition next_order)
+{
 	next_order = AIOrder::ResolveOrderPosition(vehicle_id, next_order);
 	EnforcePrecondition(false, IsValidVehicleOrder(vehicle_id, next_order));
 	return AIObject::DoCommand(0, vehicle_id, next_order, CMD_SKIP_TO_ORDER);
+}
 /**
  * Callback handler as SetOrderFlags possibly needs multiple DoCommand calls
  * to be able to set all order flags correctly. As we need to wait till the
  * command has completed before we know the next bits to change we need to
  * call the function multiple times. Each time it'll reduce the difference
  * between the wanted and the current order.
  * @param instance The AI we are doing the callback for.

src/ai/api/ai_vehicle.cpp

➞

Show inline comments

@@ @@ -186,29 +186,25 @@ @@
 	return ::Vehicle::Get(vehicle_id)->IsStoppedInDepot();
+}
 /* static */ bool AIVehicle::StartStopVehicle(VehicleID vehicle_id)
+{
 	EnforcePrecondition(false, IsValidVehicle(vehicle_id));
 	return AIObject::DoCommand(0, vehicle_id, 0, CMD_START_STOP_VEHICLE);
+}
 /* static */ bool AIVehicle::SkipToVehicleOrder(VehicleID vehicle_id, AIOrder::OrderPosition order_position)
+{
 	order_position = AIOrder::ResolveOrderPosition(vehicle_id, order_position);
 	EnforcePrecondition(false, AIOrder::IsValidVehicleOrder(vehicle_id, order_position));
 	return AIObject::DoCommand(0, vehicle_id, order_position, CMD_SKIP_TO_ORDER);
 	return AIOrder::SkipToOrder(vehicle_id, order_position);
+}
 /* static */ bool AIVehicle::ReverseVehicle(VehicleID vehicle_id)
+{
 	EnforcePrecondition(false, IsValidVehicle(vehicle_id));
 	EnforcePrecondition(false, ::Vehicle::Get(vehicle_id)->type == VEH_ROAD || ::Vehicle::Get(vehicle_id)->type == VEH_TRAIN);
 	switch (::Vehicle::Get(vehicle_id)->type) {
 		case VEH_ROAD: return AIObject::DoCommand(0, vehicle_id, 0, CMD_TURN_ROADVEH);
 		case VEH_TRAIN: return AIObject::DoCommand(0, vehicle_id, 0, CMD_REVERSE_TRAIN_DIRECTION);
 		default: NOT_REACHED();
+	}

0 comments (0 inline, 0 general)