Update of patch #4679 (project freeciv):

                  Status:               Need Info => Ready For Test         
                 Summary: Remove nreqs support entirely => Convert nreqs to
reqs on ruleset load, and only use reqs internally.

    _______________________________________________________

Follow-up Comment #3:

Updated patch, now allows rulesets to use nreqs, and silently converts them to
reqs on ruleset load.  Those portions of the patch affected by the changes
from bug #21992 have been removed.  README.effects remains unmodified, that
being addressed in patch #4401.

The iteration issue in advdata.c mentioned in comment #2 was addressed in bug
#21999.

I believe this suitable for 2.6, as it no longer requires ruleset authors to
make a hard transition, and reduces the maintenance overhead for separate
management of reqs and nreqs (and the various corner cases that might
otherwise still exist, where only reqs are examined).

(file #21073)
    _______________________________________________________

Additional Item Attachment:

File name: migrate-nreqs-to-reqs.patch    Size:9 KB


    _______________________________________________________

Reply to this item at:

  <http://gna.org/patch/?4679>

_______________________________________________
  Message sent via/by Gna!
  http://gna.org/


_______________________________________________
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev

Reply via email to