From 236abb6a6cb96caf19f8bf026046910567745d40 Mon Sep 17 00:00:00 2001 From: Darkvater Date: Tue, 13 Sep 2005 13:30:18 +0000 Subject: (svn r2949) The AI no longer needs to 'cheat' to build aircraft; eg it builds them now from a hangar. Also, to query the price of a new aircraft tile information is not needed --- aircraft_cmd.c | 34 +++++++++++++++------------------- 1 file changed, 15 insertions(+), 19 deletions(-) (limited to 'aircraft_cmd.c') diff --git a/aircraft_cmd.c b/aircraft_cmd.c index 825707dc1..14379d5b2 100644 --- a/aircraft_cmd.c +++ b/aircraft_cmd.c @@ -165,17 +165,15 @@ int32 CmdBuildAircraft(int x, int y, uint32 flags, uint32 p1, uint32 p2) if (!IsEngineBuildable(p1, VEH_Aircraft)) return CMD_ERROR; - // Workaround: TODO: make AI players try to build planes in a hangar instead of just an airport tile. - if (!IsAircraftHangarTile(tile) && IS_HUMAN_PLAYER(_current_player)) return CMD_ERROR; - - if (!IsTileOwner(tile, _current_player) && IS_HUMAN_PLAYER(_current_player)) return CMD_ERROR; - - SET_EXPENSES_TYPE(EXPENSES_NEW_VEHICLES); - value = EstimateAircraftCost(p1); + // to just query the cost, it is not neccessary to have a valid tile (automation/AI) if (flags & DC_QUERY_COST) return value; + if (!IsAircraftHangarTile(tile) || !IsTileOwner(tile, _current_player)) return CMD_ERROR; + + SET_EXPENSES_TYPE(EXPENSES_NEW_VEHICLES); + avi = AircraftVehInfo(p1); // allocate 2 or 3 vehicle structs, depending on type if (!AllocateVehicles(vl, (avi->subtype & 1) == 0 ? 3 : 2) || @@ -253,23 +251,21 @@ int32 CmdBuildAircraft(int x, int y, uint32 flags, uint32 p1, uint32 p2) _new_aircraft_id = v->index; - // the old AI doesn't click on a tile to build airplanes, so the below code will - // never work. Therefore just assume the AI's planes always come from Hangar0 - v->u.air.pos = _is_old_ai_player ? 0 : MAX_ELEMENTS; + v->u.air.pos = MAX_ELEMENTS; - /* When we click on hangar we know the tile (it is in var 'tile')it is on. By that we know - its position in the array of depots the airport has.....we can search - ->layout for #th position of depot. Since layout must start with depots, it is simple - */ + /* When we click on hangar we know the tile it is on. By that we know + * its position in the array of depots the airport has.....we can search + * layout for #th position of depot. Since layout must start with a listing + * of all depots, it is simple */ { const Station* st = GetStation(_m[tile].m2); - const AirportFTAClass* Airport = GetAirport(st->airport_type); + const AirportFTAClass* apc = GetAirport(st->airport_type); uint i; - for (i = 0; i < Airport->nof_depots; i++) { - if (st->airport_tile + ToTileIndexDiff(Airport->airport_depots[i]) == tile) { - assert(Airport->layout[i].heading == HANGAR); - v->u.air.pos = Airport->layout[i].position; + for (i = 0; i < apc->nof_depots; i++) { + if (st->airport_tile + ToTileIndexDiff(apc->airport_depots[i]) == tile) { + assert(apc->layout[i].heading == HANGAR); + v->u.air.pos = apc->layout[i].position; break; } } -- cgit v1.2.3-70-g09d2