[geos-commits] r2503 - trunk/source/operation/buffer

[svn_geos at osgeo.org]

Author: strk
Date: 2009-05-12 12:48:06 -0400 (Tue, 12 May 2009)
New Revision: 2503

Modified:
   trunk/source/operation/buffer/SubgraphDepthLocater.cpp
Log:
Have DepthSegment hold a real LineSegment, not a reference. SubgraphDepthLocater was passing the same LineSegment reference to multiple DepthSegments, messing everything up. This fixes bug #244.


Modified: trunk/source/operation/buffer/SubgraphDepthLocater.cpp
===================================================================
--- trunk/source/operation/buffer/SubgraphDepthLocater.cpp	2009-05-12 15:38:23 UTC (rev 2502)
+++ trunk/source/operation/buffer/SubgraphDepthLocater.cpp	2009-05-12 16:48:06 UTC (rev 2503)
@@ -55,7 +55,7 @@
 
 private:
 
-	geom::LineSegment& upwardSeg;
+	geom::LineSegment upwardSeg;
 
 	/*
 	 * Compare two collinear segments for left-most ordering.
@@ -80,7 +80,8 @@
 
 	int leftDepth;
 
-	DepthSegment(geom::LineSegment &seg, int depth)
+	/// @param seg will be copied to private space
+	DepthSegment(const geom::LineSegment &seg, int depth)
 		:
 		upwardSeg(seg),
 		leftDepth(depth)