[GRASS5] [bug #746] (grass) Change all G_put_map_row() -> G_put_c_raster_row() ?

Markus Neteler neteler at geog.uni-hannover.de
Tue May 29 08:51:53 EDT 2001


On Tue, May 29, 2001 at 10:32:05AM +0200, Justin Hickey via RT wrote:
> Hi Markus
> 
> mneteler wrote (Tue, May 29 2001 10:06:11):
> 
> > Subject: Change all G_put_map_row() -> G_put_c_raster_row() ?
> > 
> > grass downloaded at: CVS Server, Germany
> > 
> > There are still > 70 occurencies of the deprecated function G_put_map_row()
> > within the sources. Question is to change them to G_put_c_raster_row() for
> > proper zero/NULL handling:
> 
> Is there a G_put_raster_row(..., ..., dtatType) function? If so you should
> change the calls to G_put_raster_row(..., ..., CELL) instead. These types of
> functions modularize the code. The G_put_[cfd]_raster_row() functions should
> become static functions of whatever file they are in. That is, all calls that
> want to put a raster row should call G_put_raster_row(..., ..., dtatType)
> which in turn calls the appropriate static G_put_[cfd]_raster_row() function.

Hi Justin,

Ok, so we would have the change

G_put_map_row() ->  G_put_raster_row(..., ..., , CELL_TYPE);

If no objections, I will change all the 77 occurencies.

> My todo list includes doing this for the null_val.c since I already made some
> changes to that file. I just didn't have time to go through the code and
> change the calls yet.
> 
> Anyway, that's just my 2 cents worth.
> 
> Justin Hickey
> 
> PS Sorry about the duplicate Markus, but I don't think I can change the
> "reply to" field in RT

You can simply write to "grass5" and cc to "grass-bugs at intevation.de". Then
RT will catch the mail and add to the bug. No need to enter the RT for such
communication [this will be the test :-) ]

Cheers

 Markus




More information about the grass-dev mailing list