URL:
<http://gna.org/patch/?4645>
Summary: Remove ancient termporary kluge
Project: Freeciv
Submitted by: persia
Submitted on: Sat 05 Apr 2014 11:25:56 AM JST
Category: ai
Priority: 5 - Normal
Status: None
Privacy: Public
Assigned to: None
Originator Email:
Open/Closed: Open
Discussion Lock: Any
Planned Release:
_______________________________________________________
Details:
Syela added a kluge to prevent too many boats from collecting in
cities in r566. The need for this kluge appears to have been removed in r966,
with the introduction of ruleset-defined role definitions, but the change,
while replacing the function needing the kluge, carried over the kluge
(perhaps as a means of continuing to support non-ruleset configurations).
Over 20,000 commits later, we should probably remove this: while it
doesn't happen to do any harm with the supplied rulesets, it could cause
ruleset authors to be confused if they created a L_DEFEND_GOOD unit that
wasn't UMT_LAND, as the AI would reject these from the role.
_______________________________________________________
File Attachments:
-------------------------------------------------------
Date: Sat 05 Apr 2014 11:25:56 AM JST Name:
remove-ancient-temporary-kludge.patch Size: 1kB By: persia
<http://gna.org/patch/download.php?file_id=20477>
_______________________________________________________
Reply to this item at:
<http://gna.org/patch/?4645>
_______________________________________________
Message sent via/by Gna!
http://gna.org/
_______________________________________________
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev
