(svn r3104) -Codechange: [refit] removed now obsolite code that the old autoreplace needed
authorbjarni
Sun, 30 Oct 2005 11:00:54 +0000
changeset 2567 ee48d423f868
parent 2566 75ae28a03460
child 2568 a875eb263c49
(svn r3104) -Codechange: [refit] removed now obsolite code that the old autoreplace needed
aircraft_cmd.c
ship_cmd.c
--- a/aircraft_cmd.c	Sun Oct 30 00:16:48 2005 +0000
+++ b/aircraft_cmd.c	Sun Oct 30 11:00:54 2005 +0000
@@ -497,15 +497,12 @@
  * @param p1 vehicle ID of the aircraft to refit
  * @param p2 various bitstuffed elements
  * - p2 = (bit 0-7) - the new cargo type to refit to (p2 & 0xFF)
- * - p2 = (bit 8)   - skip check for stopped in hangar, used by autoreplace (p2 & 0x100)
- * @todo p2 bit8 check <b>NEEDS TO GO</b>
  */
 int32 CmdRefitAircraft(int x, int y, uint32 flags, uint32 p1, uint32 p2)
 {
 	Vehicle *v;
 	int pass, mail;
 	int32 cost;
-	bool SkipStoppedInHangerCheck = !!HASBIT(p2, 8); // XXX - needs to go, yes?
 	CargoID new_cid = p2 & 0xFF; //gets the cargo number
 	const AircraftVehicleInfo *avi;
 
@@ -514,7 +511,7 @@
 	v = GetVehicle(p1);
 
 	if (v->type != VEH_Aircraft || !CheckOwnership(v->owner)) return CMD_ERROR;
-	if (!SkipStoppedInHangerCheck && !CheckStoppedInHangar(v)) return_cmd_error(STR_A01B_AIRCRAFT_MUST_BE_STOPPED);
+	if (!CheckStoppedInHangar(v)) return_cmd_error(STR_A01B_AIRCRAFT_MUST_BE_STOPPED);
 
 	avi = AircraftVehInfo(v->engine_type);
 
@@ -553,10 +550,6 @@
 		u = v->next;
 		mail = (new_cid != CT_PASSENGERS) ? 0 : avi->mail_capacity;
 		u->cargo_cap = mail;
-		//autorefitted planes wants to keep the cargo
-		//it will be checked if the cargo is valid in CmdReplaceVehicle
-		if (!(SkipStoppedInHangerCheck))
-			v->cargo_count = u->cargo_count = 0;
 		v->cargo_type = new_cid;
 		InvalidateWindow(WC_VEHICLE_DETAILS, v->index);
 	}
--- a/ship_cmd.c	Sun Oct 30 00:16:48 2005 +0000
+++ b/ship_cmd.c	Sun Oct 30 11:00:54 2005 +0000
@@ -1063,25 +1063,22 @@
  * @param p1 vehicle ID of the ship to refit
  * @param p2 various bitstuffed elements
  * - p2 = (bit 0-7) - the new cargo type to refit to (p2 & 0xFF)
- * - p2 = (bit 8)   - skip check for stopped in depot, used by autoreplace (p2 & 0x100)
- * @todo p2 bit8 check <b>NEEDS TO GO</b>
  */
 int32 CmdRefitShip(int x, int y, uint32 flags, uint32 p1, uint32 p2)
 {
 	Vehicle *v;
 	int32 cost;
 	CargoID new_cid = p2 & 0xFF; //gets the cargo number
-	bool SkipStoppedInDepotCheck = !!HASBIT(p2, 8); // XXX - needs to go, yes?
 
 	if (!IsVehicleIndex(p1)) return CMD_ERROR;
 
 	v = GetVehicle(p1);
 
 	if (v->type != VEH_Ship || !CheckOwnership(v->owner)) return CMD_ERROR;
-	if (!SkipStoppedInDepotCheck) {
-		if (!IsTileDepotType(v->tile, TRANSPORT_WATER) || !(v->vehstatus&VS_STOPPED) || v->u.ship.state != 0x80)
+
+	if (!IsTileDepotType(v->tile, TRANSPORT_WATER) || !(v->vehstatus&VS_STOPPED) || v->u.ship.state != 0x80)
 			return_cmd_error(STR_980B_SHIP_MUST_BE_STOPPED_IN);
-	}
+
 
 	/* Check cargo */
 	if (!ShipVehInfo(v->engine_type)->refittable) return CMD_ERROR;
@@ -1095,10 +1092,6 @@
 	}
 
 	if (flags & DC_EXEC) {
-		//autorefitted ships wants to keep the cargo
-		//it will be checked if the cargo is valid in CmdRenewVehicle
-		if (!(SkipStoppedInDepotCheck))
-			v->cargo_count = 0;
 		v->cargo_type = new_cid;
 		InvalidateWindow(WC_VEHICLE_DETAILS, v->index);
 	}