[GRASS-dev] Re: should G_raster_map_type() use G_find_cell2() ?

GRASS GIS GRASS GIS developer
1

perhaps more to the point,

lib/gis/find_file.c

the Doxygen header comments for G_find_file() and G_find_file2()
each describe what the other function does.

?!

are the functions correct or are the comments correct?

Hamish

2

Hamish wrote:

perhaps more to the point,

lib/gis/find_file.c

the Doxygen header comments for G_find_file() and G_find_file2()
each describe what the other function does.

You're misreading the comment for G_find_file2() (easily done, due to
the odd phrasing):

* Exactly the same as G_find_file() except that if name is in the
* form nnn in ppp, and is found, name is changed to nnn by G_find_file().

This is saying: G_find_file() changes the name, but this function
doesn't.

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

3

Glynn Clements wrote:

> lib/gis/find_file.c
>
> the Doxygen header comments for G_find_file() and G_find_file2()
> each describe what the other function does.

You're misreading the comment for G_find_file2() (easily done, due to
the odd phrasing):

* Exactly the same as G_find_file() except that if name is in
* the form nnn in ppp, and is found, name is changed to nnn by
G_find_file().

This is saying: G_find_file() changes the name, but this function
doesn't.

ahh, ok. I've updated the comments in CVS to be clearer.

Hamish