diff options
author | smatz <smatz@openttd.org> | 2009-05-13 19:26:47 +0000 |
---|---|---|
committer | smatz <smatz@openttd.org> | 2009-05-13 19:26:47 +0000 |
commit | 0274886ae11171748030abb0e6cae9607494f200 (patch) | |
tree | 4ff4507d7a54e798a0cc0656135c92a1a453cf67 | |
parent | 2bc5189ad5a77fde5b418ac7dd57be290cd98d3f (diff) | |
download | openttd-0274886ae11171748030abb0e6cae9607494f200.tar.xz |
(svn r16300) -Codechange: reduce usage of Vehicle::AllocateList
-rw-r--r-- | src/aircraft_cmd.cpp | 16 | ||||
-rw-r--r-- | src/vehicle_cmd.cpp | 2 |
2 files changed, 6 insertions, 12 deletions
diff --git a/src/aircraft_cmd.cpp b/src/aircraft_cmd.cpp index a11592174..6c2ac6695 100644 --- a/src/aircraft_cmd.cpp +++ b/src/aircraft_cmd.cpp @@ -265,10 +265,8 @@ CommandCost CmdBuildAircraft(TileIndex tile, DoCommandFlag flags, uint32 p1, uin /* Prevent building aircraft types at places which can't handle them */ if (!CanVehicleUseStation(p1, GetStationByTile(tile))) return CMD_ERROR; - /* Allocate 2 or 3 vehicle structs, depending on type - * vl[0] = aircraft, vl[1] = shadow, [vl[2] = rotor] */ - Vehicle *vl[3]; - if (!Vehicle::AllocateList(vl, avi->subtype & AIR_CTOL ? 2 : 3)) { + /* We will need to allocate 2 or 3 vehicle structs, depending on type */ + if (!Vehicle::CanAllocateItem(avi->subtype & AIR_CTOL ? 2 : 3)) { return_cmd_error(STR_ERROR_TOO_MANY_VEHICLES_IN_GAME); } @@ -277,11 +275,9 @@ CommandCost CmdBuildAircraft(TileIndex tile, DoCommandFlag flags, uint32 p1, uin return_cmd_error(STR_ERROR_TOO_MANY_VEHICLES_IN_GAME); if (flags & DC_EXEC) { - Vehicle *v = vl[0]; // aircraft - Vehicle *u = vl[1]; // shadow + Vehicle *v = new Aircraft(); // aircraft + Vehicle *u = new Aircraft(); // shadow - v = new (v) Aircraft(); - u = new (u) Aircraft(); v->unitnumber = unit_num; v->direction = DIR_SE; @@ -402,9 +398,7 @@ CommandCost CmdBuildAircraft(TileIndex tile, DoCommandFlag flags, uint32 p1, uin /* Aircraft with 3 vehicles (chopper)? */ if (v->subtype == AIR_HELICOPTER) { - Vehicle *w = vl[2]; - - w = new (w) Aircraft(); + Vehicle *w = new Aircraft(); w->engine_type = p1; w->direction = DIR_N; w->owner = _current_company; diff --git a/src/vehicle_cmd.cpp b/src/vehicle_cmd.cpp index 247ec57bd..5a4c35c1f 100644 --- a/src/vehicle_cmd.cpp +++ b/src/vehicle_cmd.cpp @@ -361,7 +361,7 @@ CommandCost CmdCloneVehicle(TileIndex tile, DoCommandFlag flags, uint32 p1, uint veh_counter++; } while ((v = v->Next()) != NULL); - if (!Vehicle::AllocateList(NULL, veh_counter)) { + if (!Vehicle::CanAllocateItem(veh_counter)) { return_cmd_error(STR_ERROR_TOO_MANY_VEHICLES_IN_GAME); } } |