Opened 8 years ago
Last modified 5 years ago
#2970 new task
Replace Vect_point_in_box() by Vect_point_in_box_2d() where appropriate
| Reported by: | wenzeslaus | Owned by: | |
|---|---|---|---|
| Priority: | minor | Milestone: | 8.0.0 |
| Component: | Vector | Version: | svn-trunk |
| Keywords: | vlib, bbox, box | Cc: | |
| CPU: | Unspecified | Platform: | Unspecified |
Description
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.
Change History (8)
comment:1 by , 8 years ago
| Milestone: | 7.1.0 → 7.2.0 |
|---|
comment:3 by , 7 years ago
| Milestone: | 7.2.1 → 7.2.2 |
|---|
comment:6 by , 6 years ago
| Milestone: | → 7.2.4 |
|---|
comment:8 by , 5 years ago
| Milestone: | 7.2.4 → 8.0.0 |
|---|
I'm not aware of any changes in this, so the danger that #2734 applies for other usages of Vect_point_in_box() besides v.distance is still there.
Milestone renamed