[geos-commits] r3090 - trunk/include/geos/geom

[svn_geos at osgeo.org]

Author: strk
Date: 2010-08-23 13:38:33 +0000 (Mon, 23 Aug 2010)
New Revision: 3090

Modified:
   trunk/include/geos/geom/BinaryOp.h
Log:
Snapping heuristic was never really dropping common bits. Now it does.
This is just to have code match documentation, and still passes 'make
check' for GEOS. Wasn't tried against postgis, would be useful to do.


Modified: trunk/include/geos/geom/BinaryOp.h
===================================================================
--- trunk/include/geos/geom/BinaryOp.h	2010-08-17 18:57:38 UTC (rev 3089)
+++ trunk/include/geos/geom/BinaryOp.h	2010-08-23 13:38:33 UTC (rev 3090)
@@ -146,9 +146,12 @@
 	std::cerr<<"Computed snap tolerance: "<<snapTolerance<<std::endl;
 #endif
 
+
+#if CBR_BEFORE_SNAPPING
+	// Compute common bits
 	geos::precision::CommonBitsRemover cbr;
+	cbr.add(g0); cbr.add(g1);
 
-#if CBR_BEFORE_SNAPPING
 	// Now remove common bits
 	GeomPtr rG0( cbr.removeCommonBits(g0->clone()) );
 	GeomPtr rG1( cbr.removeCommonBits(g1->clone()) );