[GRASS-dev] making g.remove less verbose

Markus Neteler neteler at itc.it
Wed Sep 27 08:17:36 EDT 2006


On Wed, Sep 27, 2006 at 02:08:48PM +0200, Jachym Cepicky wrote:
> Hallo,
> 
> since nobody complained, i commited the "verbosity" patch for g.remove
> to cvs.
> 
> if there will be  no objections, i'll rewrite g.copy and g.rename on
> similar way
> 
> one question: in cmd/remove.c, lines 61 to 64 list of raster files is
> printed to stderr:
> 
>             G_warning(
>                _("[%s@%s] is a base map. Remove reclassed map%s first:"),
>                     name, mapset, (nrmaps > 1 ? "s" : ""));
> 
>             fprintf(stderr, " %s", *rmaps);
>             for(rmaps++; *rmaps; rmaps++)
>                 fprintf(stderr, ",%s", *rmaps);
>             fprintf(stderr, "\n");
> 
> which leds to
> 
> g.remove rast=pokus
> WARNING: [pokus at jachym] is a base map. Remove reclassed map first:
>  tmp at jachym
> 
> would it make sence, rewrite this fprintfs to G_warning too, so the
> result would be:
> 
> WARNING: [pokus at jachym] is a base map. Remove reclassed map first:
> WARNING: tmp at jachym

... this should be avoided somehow. Only one time the "WARNING"
should appear to avoid confusion.

Markus

 
> ?
> 
> jachym




More information about the grass-dev mailing list