[ET Trac] [Einstein Toolkit] #2192: schedule.ccl SYNC allows for [timelevels] suffixes
Einstein Toolkit
trac-noreply at einsteintoolkit.org
Thu Nov 8 12:39:47 CST 2018
#2192: schedule.ccl SYNC allows for [timelevels] suffixes
-------------------------+---------------------------------
Reporter: zachetie@… | Owner: Steven R. Brandt
Type: defect | Status: assigned
Priority: unset | Milestone:
Component: Other | Version: development version
Resolution: | Keywords: piraha
-------------------------+---------------------------------
Comment (by Roland Haas):
Thinking about this a bit more, while indeed STORAGE takes a group name,
it does accept expressions in square brackets afterwards, namely the
number of timelevels for which one allocates storage. How does piraha
extract those right now? Would the current patch compile and pass the test
suites?
I also notice that in the patch in line 5 the "name" pattern is
{{{
name = (?i:[a-zA-Z_][a-zA-Z0-9_\-]*\b)
}}}
ie it accepts "-" as a valid part of a name, which is not true for
variable names (since they must be valid C identifiers) and also not
really true for group names (which are alphanumeric plus the underscore).
--
Ticket URL: <https://trac.einsteintoolkit.org/ticket/2192#comment:8>
Einstein Toolkit <http://einsteintoolkit.org>
The Einstein Toolkit
More information about the Trac
mailing list