standardize DEG2RAD() etc

[hamish]

Hi,

in the source now there is

misc/m.cogo/main.c:#define DEG2RAD(a) ((a) * M_PI / 180.0)
raster/r.param.scale/param.h:     #define DEG2RAD 0.017453293
raster/wildfire/r.ros/spot_dist.c:#define DEG2RAD 0.017453292
raster/r.flow/r.flow.h:#define DEG2RAD  0.0174532925199432958

it would be good to standardize these in gis.h just following the M_PI section. i.e. add these from m.cogo:

#define DEG2RAD(a) ((a) * M_PI / 180.0)
#define RAD2DEG(a) ((a) * 180.0 / M_PI)
#define DMS2DD(d,m,s) ((d) + ((m) / 60.0) + ((s) / 3600.0))

• DMS2DD bug: if deg is negative (45:30S) -> -45 + .5 = -44.5

... or do the macros have to be in the same source file to be kept efficient?

Hamish

[hamish]

Replying to hamish:

#define DMS2DD(d,m,s) ((d) + ((m) / 60.0) + ((s) / 3600.0))

• DMS2DD bug: if deg is negative (45:30S) -> -45 + .5 = -44.5

nope, out->n_s is stored separately from out->lat in m.cogo. This is easy to forget, so dangerous to put in gis.h in hindsight.

Hamish

[neteler]

Solved in r68881 (trunk) and r68882 (relbranch72) (without DMS2DD macro)

I'd suggest wontfix for GRASS GIS 6.

[wenzeslaus]

Since this was actually fixed, closing as fixed with different milestone.

To excuse the additional noise: I think closing as solved (if possible) is good for moral in general and understanding the history in this case.

[wenzeslaus]

fixed but wontfix for 7.0 and 6.4