[GRASS-dev] Change in parser behavior?

Hamish wrote:

> since a while ago the parser has not liked option
> names with upper case characters. e.g. STREAM_DIR= in
> r.terraflow.

here we go:
http://trac.osgeo.org/grass/changeset/32263

at least for GRASS 6 should include A-Z as well.

done in 6.5svn; there is another problem though: is_option()
in parser.c does nothing if is_option() returns a 0, which
presumably leaves the arrays stranded a bit to the left and
accounts for the weirdness.

and option names can start with numbers? (ok, shrug)

Hamish

Not being a C person, I don't quite understand what this means.

Michael
____________________
C. Michael Barton, Professor of Anthropology
Director of Graduate Studies
School of Human Evolution & Social Change
Center for Social Dynamics & Complexity
Arizona State University

Phone: 480-965-6262
Fax: 480-965-7671
www: <www.public.asu.edu/~cmbarton>

On May 19, 2009, at 7:25 PM, Hamish wrote:

Hamish wrote:

since a while ago the parser has not liked option
names with upper case characters. e.g. STREAM_DIR= in
r.terraflow.

here we go:
  http://trac.osgeo.org/grass/changeset/32263

at least for GRASS 6 should include A-Z as well.

done in 6.5svn; there is another problem though: is_option()
in parser.c does nothing if is_option() returns a 0, which
presumably leaves the arrays stranded a bit to the left and
accounts for the weirdness.

and option names can start with numbers? (ok, shrug)

Hamish

Forgot to say thanks for the suggestions. We'll try out the capitalization alternatives tomorrow.

Michael
____________________
C. Michael Barton, Professor of Anthropology
Director of Graduate Studies
School of Human Evolution & Social Change
Center for Social Dynamics & Complexity
Arizona State University

Phone: 480-965-6262
Fax: 480-965-7671
www: <www.public.asu.edu/~cmbarton>

On May 19, 2009, at 7:25 PM, Hamish wrote:

Hamish wrote:

since a while ago the parser has not liked option
names with upper case characters. e.g. STREAM_DIR= in
r.terraflow.

here we go:
  http://trac.osgeo.org/grass/changeset/32263

at least for GRASS 6 should include A-Z as well.

done in 6.5svn; there is another problem though: is_option()
in parser.c does nothing if is_option() returns a 0, which
presumably leaves the arrays stranded a bit to the left and
accounts for the weirdness.

and option names can start with numbers? (ok, shrug)

Hamish

Hamish wrote:

and option names can start with numbers? (ok, shrug)

Yep; used for e.g. "g.region 3dview=...".

OTOH, r.terraflow's STREAM_DIR= was the only upper-case option name
anywhere in GRASS.

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