Opened 9 years ago
Closed 5 years ago
#3174 closed enhancement (fixed)
unit test for lwgeom_mindistance2d_tolerance
Reported by: | strk | Owned by: | nicklas |
---|---|---|---|
Priority: | medium | Milestone: | PostGIS Fund Me |
Component: | liblwgeom | Version: | master |
Keywords: | Cc: |
Description
There's currently no unit test for lwgeom_mindistance2d_tolerance except for calls with a tolerance of 0.
Adding documentation for the method would also be nice.
Change History (3)
comment:1 by , 9 years ago
comment:3 by , 5 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
Note:
See TracTickets
for help on using tickets.
I don't really get this request.
lwgeom_mindistance2d_tolerance is just an internal function with a bad name.
raising the tolerance above 0 will stop the calculations to save some cpu-cycles.
But the logic of ST_DWithin is done on the outer function LWGEOM_dwithin.
Testing if the function stops as expected I can understand, but the result will change with small changes in code without changing output.
But where to document it? and for who?