[GRASS-dev] WPS XML process description added to lib/gis/parser.c

Glynn Clements glynn at gclements.plus.com
Wed Nov 25 19:04:13 EST 2009


Soeren Gebbert wrote:

> I have added the generation of WPS process description XML documents
> to lib/gis/parser.c in grass7 .

We should probably split parser.c into multiple files. At ~84KiB, it's
easily the largest file in lib/gis.

Each of the "description" options (--help, --interface-description,
--html-description, --wps-process-description, --script) should
probably be in a separate file.

> Any hints, criticism and suggestions are welcome.

A couple of minor points regarding e.g.:

	static void wps_print_ProcessDescription()

1. Functions should use ANSI C prototypes, which means using "void"
for the argument list if the function takes no arguments.

2. We don't use CamelCase.

IOW:

	static void wps_print_process_description(void)

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


More information about the grass-dev mailing list