[GRASS-dev] [GRASS GIS] #2970: Replace Vect_point_in_box() by Vect_point_in_box_2d() where appropriate

GRASS GIS trac at osgeo.org
Fri Mar 25 07:27:30 PDT 2016

#2970: Replace Vect_point_in_box() by Vect_point_in_box_2d() where appropriate
 Reporter:  wenzeslaus       |      Owner:  grass-dev@…
     Type:  task             |     Status:  new
 Priority:  minor            |  Milestone:  7.1.0
Component:  Vector           |    Version:  svn-trunk
 Keywords:  vlib, bbox, box  |        CPU:  Unspecified
 Platform:  Unspecified      |
 In r66459 I have added `Vect_point_in_box_2d()` function which is simply a
 2D version of `Vect_point_in_box()` function which is 3D only. The
 motivation was #2734 and [https://lists.osgeo.org/pipermail/grass-
 user/2015-July/072809.html an issue reported on mailing list]. I used the
 new `Vect_point_in_box_2d()` to fix ''v.distance'' in r66460.

 `Vect_point_in_box()` function is used a lot in the modules and very often
 with hardcoded `0.0` as ''z'' parameter (and there might be also cases
 when ''z'' is optionally zero). These should be replaced by the new 2D
 only version of the function.

 An alternative would be to add one function which would optionally
 consider ''z'' using boolean ''has_z/is_2d'' or pointer for ''z'' and use
 this function in the modules.

 Currently it is not know if this is causing any issues as in case of
 ''v.distance''. Setting type to task and priority to minor.

Ticket URL: <https://trac.osgeo.org/grass/ticket/2970>
GRASS GIS <https://grass.osgeo.org>

More information about the grass-dev mailing list