#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).