Ivan Shmakov wrote: > >> I did it as %c like r.cats, other modules are more liberal and allow > >> %s. Shrug. > > > I have no use for %s just now, but there should probably be a > > GRASS-wide policy on using either %c or %s for `fs'. (In order for > > the user to learn this exactly once.) > > Since the function above allows strings, I'd opt for allowing a > string for `sep' everywhere.
When a string is used as a separator, the behaviour isn't entirely obvious. If the string is passed as the separator argument to G_tokenize(), it treats the string as a set of characters, all of which are treated as separators. For a module to provide this behaviour, it would be more natural to use e.g. "opt.sep->multiple = YES" and require the individual values to be single characters (after any parsing of backslash sequences). -- Glynn Clements <[EMAIL PROTECTED]> _______________________________________________ grass-dev mailing list [email protected] http://lists.osgeo.org/mailman/listinfo/grass-dev
