Ticket #325 (closed task: wontfix)

Opened 3 years ago

Last modified 23 months ago

there is no destroy function for CoordinateSequence

Reported by: aharon Owned by: geos-devel@…
Priority: blocker Milestone:
Component: Default Version: svn-trunk
Severity: Unassigned Keywords:
Cc:

Description

we can create CoordinateSequence? by CoordinateSequenceFactory::Creae but, there is no way to distroy the creaed CoordinteSequence?. also there is no way to create/distroy std::vector<Geomtry> .

Change History

Changed 3 years ago by aharon

we can create Coordinate Sequence by CoordinateSequenceFactory::Creae but, there is no way to distroy the creaed Coordinte Sequence. also there is no way to create/distroy std::vector<Geomtry> .

Changed 3 years ago by mloskot

I can confirm there is not named destructor provided for the CoordinateSequence? and related types. Amazingly, I have no idea myself why it's missing or even how it is supposed to be designed.

Sandro, Paul, is it intentional?

Changed 3 years ago by aharon

I think that it suppose to be in CoordinateSequenceFactory? like it is in GeometryFactory?. GeometryFactory? has createGeometry and also distroyGeometry.

what about creator for std::vector<Geometry>? it is strongly necessary.

Changed 23 months ago by strk

  • status changed from new to closed
  • resolution set to wontfix

In JTS GeometryFactory? is a class while CoordinateSequenceFactory? is an interface. In JTS neither GeometryFactory? nor CoordinateSequenceFactory? have a destroy function.

Anyway, if you feel like it'd be needed, you can provide a patch, but there's currently not much such functions would do beside simply using delete against the object, and unless you prevent directly using 'delete' I don't see much point in exposing an interface hiding that part.

Note: See TracTickets for help on using tickets.