Opened 10 years ago

Closed 6 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 nicklas, 9 years ago

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?

comment:2 by robe, 7 years ago

Milestone: PostGIS FuturePostGIS Fund Me

Milestone renamed

Note: See TracTickets for help on using tickets.