Replace Vect_point_in_box() by Vect_point_in_box_2d() where appropriate
|Reported by:||wenzeslaus||Owned by:|
|Keywords:||vlib, bbox, box||Cc:|
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 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.