Summary: Remove nreqs support entirely
Submitted by: persia
Submitted on: Wed 30 Apr 2014 01:01:42 PM JST
Priority: 5 - Normal
Status: Need Info
Assigned to: persia
Discussion Lock: Any
Planned Release: 2.6.0
As nreqs are no longer used in the default rulesets, there is no reason to
continue having engine support. The attached patch removes it all, leaving a
ruleset error on ruleset load if a ruleset uses nreqs (the network capstring
is known wrong, but I didn't want to forget adding it later).
I'm not entirely happy with the construction I used to test for the
presence of nreqs in the ruleset: suggestions for a better implementation (not
using reqs->size, which ought be private) welcomed.
This patch has not been extensively tested: it is posted more for review
for completeness and verification of the intent than as a direct commit
candidate. I'm confident there will need to be changes before application
(for example, I doubt I removed enough code from ruledit).
Date: Wed 30 Apr 2014 01:01:42 PM JST Name: remove-nreqs-support.patch Size:
17kB By: persia
Reply to this item at:
Message sent via/by Gna!
Freeciv-dev mailing list