Update of bug #16676 (project freeciv):
Status: In Progress = Ready For Test
Planned Release: = 2.3.0
___
Follow-up Comment #5:
Patch with
Additional Item Attachment, bug #16676 (project freeciv):
File name: trunk_remove_player_ai_est_upkeep.diff Size:7 KB
___
Reply to this item at:
http://gna.org/bugs/?16676
___
Message
Update of bug #16676 (project freeciv):
Status: Ready For Test = In Progress
___
Follow-up Comment #4:
ai_calc_data() does not have effect because it computes the pointers instead
of the _trade_ and
Follow-up Comment #3, bug #16676 (project freeciv):
two patches to remove both variables:
* prev_gold: not used anywhere
* est_upkeep: used in the two mentioned files but nowhere calculated; add
aihand.c:ai_calc_data() to calculate the expenses and use this value. I will
update bug #16677 once
Update of bug #16676 (project freeciv):
Status:None = Ready For Test
Assigned to:None = pepeto
___
Reply to this item at:
Follow-up Comment #2, bug #16676 (project freeciv):
_est_upkeep_ is still used in aicity.c and aidiplomat.c, but not sure it has
been computed somewhere.
___
Reply to this item at:
http://gna.org/bugs/?16676
URL:
http://gna.org/bugs/?16676
Summary: unused variables in struct player_ai
Project: Freeciv
Submitted by: syntron
Submitted on: Freitag 10.09.2010 um 21:35
Category: general
Severity: 3 - Normal
Follow-up Comment #1, bug #16676 (project freeciv):
They looks like old fields. _prev_gold_ has been added in initial revision
(rev #77), and _est_upkeep_ at revision #1821. I guess we can remove them.
___
Reply to this item at: