CST hangs if scheduling block isn't complete

[Ian Hinder]

If a schedule.ccl file contains

schedule group A in A

the CST hangs at "Creating schedule bindings...". This error in the schedule.ccl file should be detected and a fatal error message should be generated.

Keyword:

[Frank Löffler]

The problem isn't the recursion (alone), but the missing {}"" block after this. the following shows the same:

schedule group A in B

while this doesn't hang:

schedule group A in A { } ""

I believe cyclic dependencies are checked at runtime only, but I didn't check A->A

[Frank Löffler]

Please review the attached patch which solves the problem for me. I request two 'yes' because of the sensitive place (CST).

[Erik Schnetter]

Scheduling a group in itself is possible in Cactus. One has to disable printing the schedule, though. This does make sense if there is an IF or a WHILE modifier.

[Frank Löffler]

This patch doesn't change the ability to schedule something within itself. It only fixes the hang, which is unrelated to this. The patch is really simple: it is a one-line change checking whether the end of the file has been reached before trying (an otherwise always) failing regexp. I still need to "yes".

[Roland Haas]

Patch is fine by me.

[Erik Schnetter]

Please apply.