[GRASS5] GUI code cleanup patch

Glynn Clements glynn at gclements.plus.com
Mon Mar 13 01:18:09 EST 2006


Cedric Shock wrote:

> This patch cleans up the calling convention between lib/gis/parser.c and 
> lib/gis/gui.tcl. It extends general/g.parser/main.c to cover all of the 
> string attributes available via the c interface. It adds a new char * 
> guisection attribute to options and flags in include/gis.h.
> 
> In detail:
> 
> Flag and module attributes are now passed from lib/gis/parser.c to 
> lib/gis/gui.tcl as the flattened list form of a tcl array, just like the 
> attributes for options.

That seems reasonable.

> I moved GUI logic regarding text to use for labels (label or description) from 
> lib/gis/parser.c to lib/gis/gui.tcl. This will allow a gui to decide, for 
> example, to display the description text as "tooltip" or balloon help when an 
> option has a label.

That seems reasonable, except that I would suggest leaving the
label/desc entries untouched, and either moving the choice to the
point where the value is actually used, or creating a separate entry
in opts() and copying the label/desc entry to that instead.

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




More information about the grass-dev mailing list