URL:
<http://gna.org/patch/?4407>
Summary: Move requirement range/survives sanity checking from
req_from_str() to rssanity.c
Project: Freeciv
Submitted by: jtn
Submitted on: Sat Jan 4 19:24:19 2014
Category: None
Priority: 5 - Normal
Status: None
Privacy: Public
Assigned to: None
Originator Email:
Open/Closed: Open
Discussion Lock: Any
Planned Release: 2.6.0
_______________________________________________________
Details:
Currently, req_from_str() does some sanity checking of requirements when
parsing them: it checks validity of ranges for requirements, and (after patch
#4396) the 'survives' field.
cazfi notes in bug #21418 comment 1:
> On a general note, we need to be moving all possible sanity
> checks to sanity_check_ruleset_data() which freeciv-ruledit then
> can use to check ruleset being modified.
req_from_str() should be reduced to checking the syntax of individual fields,
and semantic checking of ranges vs types etc moved to rssanity.c (probably
building on bug #21418).
(I think this will lose us reporting of which filename/section is at fault,
but it can't be helped.)
_______________________________________________________
Reply to this item at:
<http://gna.org/patch/?4407>
_______________________________________________
Message sent via/by Gna!
http://gna.org/
_______________________________________________
Freeciv-dev mailing list
[email protected]
https://mail.gna.org/listinfo/freeciv-dev
