[ET Trac] [Einstein Toolkit] #1194: Cactus does not check range of integer parameters anymore
Einstein Toolkit
trac-noreply at einsteintoolkit.org
Sun Dec 9 20:13:44 CST 2012
#1194: Cactus does not check range of integer parameters anymore
-------------------+--------------------------------------------------------
Reporter: rhaas | Type: defect
Status: new | Priority: major
Milestone: | Component: Cactus
Version: | Keywords:
-------------------+--------------------------------------------------------
since adding the possibility to use formula on the rhs of parameter value
assignments in parfiles (rev r4797 of the flesh) a bug is present that
prevents Cactus from rejecting CCTK_INT parameter values that are not in
the allowed range. Instead the default value is used. Christian Ott
encountered this today. The bug is caused by not re-initializing retval
before entering a loop in src/main/ParameterSetInterger (compare
ParameterSetDouble).
The attached patch fixes this and the second patch (to Exact) fixes a typo
in the test parameter files that was not caught because of this bug (test
results do not change).
When parameters within the allowed range were supplied the code produced
the right parameter values, but incorrect parameters would have silently
appeared to the code as the default value.
I would like to also apply this to the release.
--
Ticket URL: <https://trac.einsteintoolkit.org/ticket/1194>
Einstein Toolkit <http://einsteintoolkit.org>
The Einstein Toolkit
More information about the Trac
mailing list