<URL: http://bugs.freeciv.org/Ticket/Display.html?id=40148 >
On 12/03/2008, Marko Lindqvist wrote:
>
> Running S2_2 r14463 autogames, almost all games have
> 1: handle_unit_upgrade() invalid unit
> errors
AI was trying to upgrade allied units. Fix attached.
- ML
diff -Nurd -X.diff_ignore freeciv/ai/aicity.c freeciv/ai/aicity.c
--- freeciv/ai/aicity.c 2008-03-10 20:08:04.000000000 +0200
+++ freeciv/ai/aicity.c 2008-03-27 04:09:53.000000000 +0200
@@ -1474,32 +1474,35 @@
struct player *pplayer = city_owner(pcity);
unit_list_iterate(pcity->tile->units, punit) {
- struct unit_type *punittype = can_upgrade_unittype(pplayer, unit_type(punit));
-
- if (military && !IS_ATTACKER(punit)) {
- /* Only upgrade military units this round */
- continue;
- } else if (!military && IS_ATTACKER(punit)) {
- /* Only civilians or tranports this round */
- continue;
- }
- if (punittype) {
- int cost = unit_upgrade_price(pplayer, unit_type(punit), punittype);
- int real_limit = limit;
+ if (pcity->owner == punit->owner) {
+ /* Only upgrade units you own, not allied ones */
+ struct unit_type *punittype = can_upgrade_unittype(pplayer, unit_type(punit));
- /* Triremes are DANGEROUS!! We'll do anything to upgrade 'em. */
- if (unit_has_type_flag(punit, F_TRIREME)) {
- real_limit = pplayer->ai.est_upkeep;
+ if (military && !IS_ATTACKER(punit)) {
+ /* Only upgrade military units this round */
+ continue;
+ } else if (!military && IS_ATTACKER(punit)) {
+ /* Only civilians or tranports this round */
+ continue;
}
- if (pplayer->economic.gold - cost > real_limit) {
- CITY_LOG(LOG_BUY, pcity, "Upgraded %s to %s for %d (%s)",
- unit_rule_name(punit),
- utype_rule_name(punittype),
- cost,
- military ? "military" : "civilian");
- handle_unit_upgrade(city_owner(pcity), punit->id);
- } else {
- increase_maxbuycost(pplayer, cost);
+ if (punittype) {
+ int cost = unit_upgrade_price(pplayer, unit_type(punit), punittype);
+ int real_limit = limit;
+
+ /* Triremes are DANGEROUS!! We'll do anything to upgrade 'em. */
+ if (unit_has_type_flag(punit, F_TRIREME)) {
+ real_limit = pplayer->ai.est_upkeep;
+ }
+ if (pplayer->economic.gold - cost > real_limit) {
+ CITY_LOG(LOG_BUY, pcity, "Upgraded %s to %s for %d (%s)",
+ unit_rule_name(punit),
+ utype_rule_name(punittype),
+ cost,
+ military ? "military" : "civilian");
+ handle_unit_upgrade(city_owner(pcity), punit->id);
+ } else {
+ increase_maxbuycost(pplayer, cost);
+ }
}
}
} unit_list_iterate_end;
_______________________________________________
Freeciv-dev mailing list
[email protected]
https://mail.gna.org/listinfo/freeciv-dev
