[postgis-tickets] r14438 - Reduce paranoia in ST_MakeValid

Sandro Santilli strk at keybit.net
Thu Nov 26 05:00:22 PST 2015


Author: strk
Date: 2015-11-26 05:00:21 -0800 (Thu, 26 Nov 2015)
New Revision: 14438

Modified:
   trunk/liblwgeom/lwgeom_geos_clean.c
Log:
Reduce paranoia in ST_MakeValid

Modified: trunk/liblwgeom/lwgeom_geos_clean.c
===================================================================
--- trunk/liblwgeom/lwgeom_geos_clean.c	2015-11-26 09:15:07 UTC (rev 14437)
+++ trunk/liblwgeom/lwgeom_geos_clean.c	2015-11-26 13:00:21 UTC (rev 14438)
@@ -44,6 +44,7 @@
 #include <assert.h>
 
 /* #define POSTGIS_DEBUG_LEVEL 4 */
+/* #define PARANOIA_LEVEL 2 */
 #undef LWGEOM_PROFILE_MAKEVALID
 
 
@@ -951,6 +952,7 @@
 	}
 	}
 
+#if PARANOIA_LEVEL > 1
 	/*
 	 * Now check if every point of input is also found
 	 * in output, or abort by returning NULL
@@ -958,10 +960,6 @@
 	 * Input geometry was lwgeom_in
 	 */
 	{
-		const int paranoia = 2;
-		/* TODO: check if the result is valid */
-		if (paranoia)
-		{
 			int loss;
 			GEOSGeometry *pi, *po, *pd;
 
@@ -981,8 +979,8 @@
 				lwnotice("Vertices lost in LWGEOM_GEOS_makeValid");
 				/* return NULL */
 			}
-		}
 	}
+#endif /* PARANOIA_LEVEL > 1 */
 
 
 	return gout;



More information about the postgis-tickets mailing list