[GRASS-dev] Raster lib in GRASS7
glynn at gclements.plus.com
Fri Jun 19 22:45:21 EDT 2009
Martin Landa wrote:
> >> >> I started a wiki page dedicated to the raster lib in GRASS7. The page
> >> >> currently contains initial list of functions which should be moved
> >> >> from libGIS.
> >> >>
> >> >> http://trac.osgeo.org/grass/wiki/Grass7RasterLib
> >> >>
> >> >> Logically raster lib should be placed in 'lib/raster'. Currently there
> >> >> are R_() functions. Where should be moved content of 'lib/raster'?
> >> >
> >> > I suppose that it can be merged into lib/display, as that's mostly
> >> > what uses it.
> >> I don't know display library so well, but I would guess that the
> >> merging cannot be done automatically (just rename R_ to D_). I have
> >> created for this purpose wiki page, feel free to updated it
> > I don't suggest renaming the functions, just moving raster.c to
> > lib/display.
> OK, but shouldn't be better to use the same prefix for all fns from
> the display library?
Many of the R_* functions have matching D_* functions with the same
name (modulo the prefix) and "matching" functionality.
OTOH, many of the R_* functions no longer need to be externally
visible. AFAICT, only the following are used other than by
Everything else can be made "static" (or eliminated; most of them are
just aliases for the corresponding COM_* functions in lib/driver).
AFAICT, all of the above except for R_erase() can be changed to D_*;
none of the other R_* functions which have matching D_* versions are
Glynn Clements <glynn at gclements.plus.com>
More information about the grass-dev