Opened 9 years ago

Closed 9 years ago

#2632 closed patch (fixed)

ST_AsGML for curve types

Reported by: mhugent Owned by: pramsey
Priority: medium Milestone: PostGIS 2.2.0
Component: postgis Version: master
Keywords: Cc: pka

Description

Dear PostGIS team

Find attached a patch adding GML output for the curve types (circularstring, compoundcurve, curvepolygon, multicurve, multisurface). Cunit and sql tests are provided as well. As it is my second PostGIS contribution (after 7 years break), please review carefully.

Thanks, Marco

Attachments (1)

140204_as_gml_curves.diff (35.9 KB ) - added by mhugent 9 years ago.

Download all attachments as: .zip

Change History (5)

by mhugent, 9 years ago

Attachment: 140204_as_gml_curves.diff added

comment:1 by pka, 9 years ago

Cc: pka added

comment:2 by pramsey, 9 years ago

Looks pretty good to me, thanks for including tests, etc. I'll have to see if the team is OK w/ a 2.1 add, it's new functionality, though not a new function signature, so perhaps could be added to 2.1.

comment:3 by robe, 9 years ago

Milestone: PostGIS 2.1.2PostGIS 2.2.0
Version: 2.1.xtrunk

I think the code change is too big to risk going into a micro. Besides we don't release new functionaltiy in a micro because its too hard to explain to people who rely on new functionality — use 2.1.2 or higher (much easier to say use 2.2)

comment:4 by pramsey, 9 years ago

Resolution: fixed
Status: newclosed

Trunk at r12227

Note: See TracTickets for help on using tickets.