Enforce that arrangement names can only contain letters, numbers, or underscores

Comments (4)

1. 

   Frank Löffler

   • removed comment

   Why should we do this? What about, for example, dashes, tildes, spaces, dots? What about umlauts? How should we enforce the non-use of some characters?

   I can understand that a project like the Einstein Toolkit should avoid certain characters like backslashes or colons, which is a strategical decision. I disagree that Cactus itself should limit that choice technically. On the contrary, it should work with whatever it finds.

   • 2012-09-10T13:02:20+00:00
2. 

   Erik Schnetter reporter

   • removed comment

   This restriction is described in the users' guide.

   Enforcing means that we check all arrangement names in the thorn list, and abort with an error if an illegal character is found.

   Technically, C identifiers are created from arrangement names, after converting them to upper case. Having e.g. dashes in there leads to very strange error messages, if at all. See CST, routine CreateThornsHeaders, and the auto-generated file cctk_DefineThorn.h.

   • 2012-09-10T19:16:29+00:00
3. 

   Erik Schnetter reporter

   • changed status to resolved
   • removed comment

   Since restricting arrangement names to be legal C identifiers is currently required by Cactus, I have added a respective check where the thorn name is check as well (file CST, routine CreateThornList).

   • 2012-09-16T18:34:03+00:00
4. 

   Roland Haas

   • edited description
   • changed status to closed

   • 2019-02-21T19:52:45+00:00
5. Log in to comment