Opened 8 years ago

Closed 5 years ago

#6187 closed enhancement (wontfix)

Add support for MultiPoint geometry to Get/Set coordinate functions

Reported by: oleinik Owned by: warmerdam
Priority: normal Milestone: closed_because_of_github_migration
Component: OGR_SF Version: 2.0.1
Severity: normal Keywords:
Cc:

Description (last modified by oleinik)

Please add support for MultiPoint/MultiPoint25D to OGR_G_SetPoints, OGR_G_GetPoints, OGR_G_GetPointsCount and other methods for getting/setting vertices in points, line strings and rings. It's will simplify access to elements of such collection geometry.

Change History (7)

comment:1 by oleinik, 8 years ago

Description: modified (diff)

comment:2 by Even Rouault, 8 years ago

Patch welcome

comment:3 by meier, 8 years ago

There is support in the current API, check out addPoint http://www.gdal.org/classOGRSimpleCurve.html and its variants addPointM() etc. There is also the option of getting number of vertices for the said geometries. getNumGeometries() is also there in the trunk version, not in the releases yet, so you might want to consider that too.

Last edited 8 years ago by meier (previous) (diff)

in reply to:  3 comment:4 by Even Rouault, 8 years ago

Replying to meier:

There is support in the current API, check out addPoint http://www.gdal.org/classOGRSimpleCurve.html and its variants addPointM() etc. There is also the option of getting number of vertices for the said geometries. getNumGeometries() is also there in the trunk version, not in the releases yet, so you might want to consider that too.

The request from the reporter is about extending the C API to deal with multipoints. The reference to OGRSimpleCurve isn't really relevant since OGRMultiPoint doesn't derive from it.

comment:5 by meier, 8 years ago

Right you are, I didn't notice the C API reference. @oleinik, even so, the C API does support these methods, check out OGR_G_AddGeometry(), OGR_G_SetPoint (), OGR_G_GetPoint (), OGR_G_GetPointCount() etc on http://www.gdal.org/ogr__api_8h.html.

Last edited 8 years ago by meier (previous) (diff)

comment:6 by Jukka Rahkonen, 6 years ago

@meier says, if I understand it right, that this ticket is invalid. Are there other opinions? Are these methods poorly documented?

comment:7 by Even Rouault, 5 years ago

Milestone: closed_because_of_github_migration
Resolution: wontfix
Status: newclosed

This ticket has been automatically closed because Trac is no longer used for GDAL bug tracking, since the project has migrated to GitHub. If you believe this ticket is still valid, you may file it to https://github.com/OSGeo/gdal/issues if it is not already reported there.

Note: See TracTickets for help on using tickets.