[GRASS-dev] [grass-code I][458] 6.2 modules using new parser flags

code I item #458, was opened at 2007-08-11 16:28
Status: Open
Priority: 2
Submitted By: Scott Mitchell (smitch)
Assigned to: Nobody (None)

Summary: 6.2 modules using new parser flags

Issue type: module bug
Issue status: fixed in CVS
GRASS version: 6.2
GRASS component: display

Operating system: all

Operating system version:
GRASS CVS checkout date, if applies (YYMMDD):

Comment By: Maciej Sieczka (msieczka)

Date: 2007-08-11 22:44

Message:
I have checked current 6.2 CVS source and I can find no occurences of
--q or --v flags in use by any modules.

BTW however, I have noticed that GEM already uses it's own --q flag
(for --query, not for --quiet). I'm curious if this could cause any
problems for the parser and if it should be changed. Apllies to both
6.2 and 6.3.

----------------------------------------------------------------------

You can respond by visiting:
http://wald.intevation.org/tracker/?func=detail&atid=204&aid=458&group_id=21

Maciej Sieczka wrote:

code I item #458, was opened at 2007-08-11 16:28
Status: Open
Priority: 2
Submitted By: Scott Mitchell (smitch)
Assigned to: Nobody (None)
>Summary: 6.2 modules using new parser flags
Issue type: module bug
Issue status: fixed in CVS
GRASS version: 6.2
GRASS component: display
>Operating system: all
Operating system version:
GRASS CVS checkout date, if applies (YYMMDD):

>Comment By: Maciej Sieczka (msieczka)
Date: 2007-08-11 22:44

Message:
I have checked current 6.2 CVS source and I can find no occurences of
--q or --v flags in use by any modules.

The --q[uiet] and --v[erbose] flags are implemented by G_parser(), and
aren't present in 6.2.x.

I'm unsure whether to remove the --quiet flags only from the 6.2.x
version of d.slide.show, or also from the 6.3.x version.

IMHO, d.slide.show should simply allow the d.rast/d.vect commands to
operate at the verbosity level set by $GRASS_VERBOSE. If d.slide.show
itself is passed --q or --v, GRASS_VERBOSE will be set accordingly,
and the d.rast/d.vect commands will inherit that setting. Otherwise,
they will inherit any existing setting of $GRASS_VERBOSE.

Forcing the commands to operate quietly regardless of --v or any
$GRASS_VERBOSE setting seems incorrect.

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

Glynn Clements wrote:

Maciej Sieczka wrote:

code I item #458, was opened at 2007-08-11 16:28
Status: Open
Priority: 2
Submitted By: Scott Mitchell (smitch)
Assigned to: Nobody (None)

Summary: 6.2 modules using new parser flags

Issue type: module bug
Issue status: fixed in CVS
GRASS version: 6.2
GRASS component: display

Operating system: all

Operating system version:
GRASS CVS checkout date, if applies (YYMMDD):

Comment By: Maciej Sieczka (msieczka)

Date: 2007-08-11 22:44

Message:
I have checked current 6.2 CVS source and I can find no occurences of
--q or --v flags in use by any modules.

The --q[uiet] and --v[erbose] flags are implemented by G_parser(), and
aren't present in 6.2.x.

I meant that I checked if, in error, any of them was used in other 6.2
modules besides the d.slide.show.

Maciek