Opened 4 years ago

Closed 5 months ago

#3326 closed enhancement (wontfix)

Drop liblwgeom_internal.h use from postgis module

Reported by: strk Owned by: strk
Priority: medium Milestone: PostGIS 3.0.0
Component: liblwgeom Version: master
Keywords: Cc:

Description

This is propedeutic to giving liblwgeom his own release cycles.

Change History (7)

comment:1 Changed 4 years ago by strk

It currently seems to be only needed for FP comparison and a single function that might be good to expose to the public liblwgeom API:

geography_btree.c:#include "liblwgeom_internal.h"         /* For FP comparators. */
geography_measurement.c:#include "liblwgeom_internal.h"         /* For FP comparators. */
lwgeom_functions_analytic.c:#include "liblwgeom_internal.h"  /* For FP comparators. */
lwgeom_in_geohash.c:#include "liblwgeom_internal.h"/* for decode_geohash_bbox */
lwgeom_rtree.c:#include "liblwgeom_internal.h"         /* For FP comparators. */

comment:2 Changed 3 years ago by robe

Component: postgisliblwgeom
Owner: changed from pramsey to strk

strk you still want to do this, or punt to 2.4.0?

comment:3 Changed 3 years ago by strk

Punt please

comment:4 Changed 3 years ago by robe

Milestone: PostGIS 2.3.0PostGIS 2.4.0

comment:5 Changed 2 years ago by robe

Milestone: PostGIS 2.4.0PostGIS 2.5.0

comment:6 Changed 17 months ago by komzpa

Milestone: PostGIS 2.5.0PostGIS 3.0.0

comment:7 Changed 5 months ago by Algunenano

Resolution: wontfix
Status: newclosed

This no longer makes sense as any .h can be considered internal and won't be installed.

Note: See TracTickets for help on using tickets.