Skip to content

Commit

Permalink
Be more explicit about memory management in noder
Browse files Browse the repository at this point in the history
  • Loading branch information
strk committed Jul 25, 2020
1 parent 0f93897 commit 5586055
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion include/geos/noding/Noder.h
Original file line number Diff line number Diff line change
Expand Up @@ -55,14 +55,17 @@ class GEOS_DLL Noder {
* others may only add some or none at all.
*
* @param segStrings a collection of {@link SegmentString}s to node
* The caller remains responsible for releasing the memory
* associated with the container and its elements.
*/
virtual void computeNodes(std::vector<SegmentString*>* segStrings) = 0;

/** \brief
* Returns a collection of fully noded [SegmentStrings](@ref SegmentString).
* The SegmentStrings have the same context as their parent.
*
* @return a newly allocated std::vector of const SegmentStrings.
* @return a newly allocated std::vector of newly allocated
* SegmentStrings (copies of input, if needs be).
* Caller is responsible to delete container and elements.
*/
virtual std::vector<SegmentString*>* getNodedSubstrings() const = 0;
Expand Down

0 comments on commit 5586055

Please sign in to comment.