Update of patch #2963 (project freeciv):
Status: Ready For Test => In Progress
_______________________________________________________
Follow-up Comment #9:
- When considering value of building caravan, consider ruleset set traderoute
type trade % instead of blindly assuming that intercontinental trade is better
than continental.
- Removed wrapper function aiferry_boats_available() that did nothing but
called aiferry_avail_boats() Call latter directly instead
- Minor style corrections
Still needs to decide what to do with dai_is_unit_tired_waiting_boat(). It's
counting time from caravan's original birth. Old units could not be reassigned
to a mission, as they would give up immediately.
I wonder why there's "if (unit_transported(punit) ||
is_ocean_tile(unit_tile(punit)))" when checking if unit needs a boat. Is the
latter part meant to check if unit can itself move on ocean tiles? (if it's
not transported as first checked, then to exist in ocean tile it must be able
to live there itself - ignoring possibility of ocean cities, native bases, and
native roads) In any case this cannot be right.
(file #16353)
_______________________________________________________
Additional Item Attachment:
File name: ContTrade-2.patch.bz2 Size:6 KB
_______________________________________________________
Reply to this item at:
<http://gna.org/patch/?2963>
_______________________________________________
Message sent via/by Gna!
http://gna.org/
_______________________________________________
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev
