Opened 12 years ago

Closed 5 years ago

#1941 closed patch (fixed)

supporting lwgeom_grid() in liblwgeom

Reported by: esseffe Owned by: strk
Priority: medium Milestone: PostGIS Fund Me
Component: liblwgeom Version: master
Keywords: Cc: esseffe

Description

I notice that lwgeom_grid() isn't available on liblwgeom; it's currently implemented directcly into the PostGIS main-core. IMHO liblwgeom looks to be a more appropriate place for this API.

the attached patch allows to transfer lwgeom_grid() directly in liblwgeom.

bye Sandro Furieri

Attachments (1)

lwgeom_grid_patch​ (20.0 KB ) - added by esseffe 12 years ago.

Download all attachments as: .zip

Change History (9)

by esseffe, 12 years ago

Attachment: lwgeom_grid_patch​ added

comment:1 by strk, 12 years ago

One thing Sandro: anything going into the liblwgeom.h file (public API) would better be prefixed by lw, so gridspec should be lw_gridspec or something like that.

comment:2 by robe, 12 years ago

Cc: esseffe added

Sandro does Sandro need to provide another patch? Wasn't absolutely clear from your comment.

comment:3 by robe, 12 years ago

Type: enhancement → patch

comment:4 by strk, 12 years ago

I'd appreciate a new patch, yes. @esseffe are you going to provide it ?

comment:5 by robe, 12 years ago

Milestone: PostGIS 2.1.0 → PostGIS 2.2.0

comment:6 by strk, 10 years ago

Milestone: PostGIS 2.2.0 → PostGIS Future

comment:7 by robe, 7 years ago

Milestone: PostGIS Future → PostGIS Fund Me

Milestone renamed

comment:8 by Algunenano, 5 years ago

Resolution: → fixed
Status: new → closed

This was done at some point in the past.

Note: See TracTickets for help on using tickets.