[GRASS-dev] G_legal_filename and "/"

[Glynn Clements]

J-Báchym Čepický wrote:-A

> IMHO "/" character is legal character in file name. I suggest change
> line 43 in lib/gis/legal_filename.c:
> 
> 
> -	if (*s == '/' || *s == '"' || *s == '\'' || *s <= ' ' ||
> 
> +	if (*s == '"' || *s == '\'' || *s <= ' ' ||
> 
> What do you think?

No, "/" most definitely isn't a legal character in a filename, as it's
used as a directory separator on all supported platforms. It's legal
in a *pathname*, but that isn't what G_legal_filename() is meant for.

-- 
Glynn Clements <glynn at gclements.plus.com>