Opened 13 years ago

Last modified 10 years ago

#477 closed enhancement

Add GEOSCoordSeq_set function that takes a list of coordinates — at Version 3

Reported by: olt Owned by: hobu
Priority: minor Milestone: 3.5.0
Component: C API Version: main
Severity: Unassigned Keywords:
Cc: strk

Description (last modified by hobu)

Creating a GEOSCoordSequence requires calls to GEOSCoordSeq_setX and GEOSCoordSeq_setY for each point. There should be a GEOSCoordSeq_set function that takes a list of points.

Something like:

GEOSCoordSeq_set(const GEOSCoordSequence* s, double *val, unsigned int n)

Change History (4)

comment:1 by hobu, 13 years ago

Cc: strk added
Description: modified (diff)

comment:2 by strk, 13 years ago

I don't understand that signature. Is it an array of pointers to doubles ? Does it save 1 call for point or is it meant to do 1 call per dimension for any sized CoordinateSequence ?

comment:3 by hobu, 13 years ago

Description: modified (diff)

Attached is a patch that starts the attempt. I am unable to compile it because of the Lion ar/llvm bug https://svn.macports.org/ticket/30309

by hobu, 13 years ago

Attachment: 447-patch.patch added
Note: See TracTickets for help on using tickets.