From 6f010e5c9b72b6134b94b76b23ef75e90e1eee3d Mon Sep 17 00:00:00 2001
From: Sandro Santilli <strk@keybit.net>
Date: Fri, 15 Oct 2004 15:00:56 +0000
Subject: [PATCH] Added debugging lines

git-svn-id: http://svn.osgeo.org/postgis/trunk@1013 b70326c6-7e19-0410-871a-916f4a2858ee
---
 lwgeom/lwgeom_functions_basic.c | 52 ++++++++++++++++++++++++++++-----
 1 file changed, 45 insertions(+), 7 deletions(-)

diff --git a/lwgeom/lwgeom_functions_basic.c b/lwgeom/lwgeom_functions_basic.c
index 325065ebc..6254cbbef 100644
--- a/lwgeom/lwgeom_functions_basic.c
+++ b/lwgeom/lwgeom_functions_basic.c
@@ -1576,23 +1576,32 @@ Datum LWGEOM_accum(PG_FUNCTION_ARGS)
 	Pointer **pointers;
 	MemoryContext oldcontext;
 
-//elog(NOTICE, "LWGEOM_accum called");
+#ifdef DEBUG
+	elog(NOTICE, "LWGEOM_accum called");
+#endif
 
 	datum = PG_GETARG_DATUM(0);
 	if ( (Pointer *)datum == NULL ) {
 		array = NULL;
 		nelems = 0;
-		//elog(NOTICE, "geom_accum: NULL array, nelems=%d", nelems);
+#ifdef DEBUG
+		elog(NOTICE, "geom_accum: NULL array");
+#endif
 	} else {
 		array = (ArrayType *) PG_DETOAST_DATUM_COPY(datum);
 		nelems = ArrayGetNItems(ARR_NDIM(array), ARR_DIMS(array));
+#ifdef DEBUG
+		elog(NOTICE, "geom_accum: array of nelems=%d", nelems);
+#endif
 	}
 
 	datum = PG_GETARG_DATUM(1);
 	// Do nothing, return state array
 	if ( (Pointer *)datum == NULL )
 	{
-		//elog(NOTICE, "geom_accum: NULL geom, nelems=%d", nelems);
+#ifdef DEBUG
+		elog(NOTICE, "geom_accum: NULL geom, nelems=%d", nelems);
+#endif
 		PG_RETURN_ARRAYTYPE_P(array);
 	}
 
@@ -1607,8 +1616,6 @@ Datum LWGEOM_accum(PG_FUNCTION_ARGS)
 	/* Make a DETOASTED copy of input geometry */
 	geom = (PG_LWGEOM *)PG_DETOAST_DATUM_COPY(datum);
 
-	//elog(NOTICE, "geom_accum: adding %p (nelems=%d)", geom, nelems);
-
 	/*
 	 * Might use a more optimized version instead of lwrealloc'ing
 	 * at every iteration. This is not the bottleneck anyway.
@@ -1621,11 +1628,18 @@ Datum LWGEOM_accum(PG_FUNCTION_ARGS)
 		result->size = nbytes;
 		result->ndim = 1;
 		*((int *) ARR_DIMS(result)) = nelems;
+#ifdef DEBUG
+		elog(NOTICE, "geom_accum: adding %p (nelems=%d)", geom, nelems);
+#endif
+
 	} else {
 		result = (ArrayType *) lwrealloc(array, nbytes);
 		result->size = nbytes;
 		result->ndim = 1;
 		*((int *) ARR_DIMS(result)) = nelems;
+#ifdef DEBUG
+		elog(NOTICE, "geom_accum: adding %p (nelems=%d)", geom, nelems);
+#endif
 	}
 
 	pointers = (Pointer **)ARR_DATA_PTR(result);
@@ -1661,7 +1675,9 @@ Datum LWGEOM_collect_garray(PG_FUNCTION_ARGS)
 	unsigned int outtype;
 	int i;
 
-//elog(NOTICE, "LWGEOM_collect_garray called");
+#ifdef DEBUG
+	elog(NOTICE, "LWGEOM_collect_garray called");
+#endif
 
 	/* Get input datum */
 	datum = PG_GETARG_DATUM(0);
@@ -1679,6 +1695,10 @@ Datum LWGEOM_collect_garray(PG_FUNCTION_ARGS)
 	/* Get number of geometries in array */
 	nelems = ArrayGetNItems(ARR_NDIM(array), ARR_DIMS(array));
 
+#ifdef DEBUG
+	elog(NOTICE, "LWGEOM_collect_garray: array has %d elements", nelems);
+#endif
+
 	/* Return null on 0-elements input array */
 	if ( nelems == 0 )
 	{
@@ -1756,7 +1776,9 @@ Datum LWGEOM_makeline_garray(PG_FUNCTION_ARGS)
 	unsigned int npoints;
 	int i;
 
-//elog(NOTICE, "LWGEOM_makeline_garray called");
+#ifdef DEBUG
+	elog(NOTICE, "LWGEOM_makeline_garray called");
+#endif
 
 	/* Get input datum */
 	datum = PG_GETARG_DATUM(0);
@@ -1771,9 +1793,17 @@ Datum LWGEOM_makeline_garray(PG_FUNCTION_ARGS)
 	/* Get actual ArrayType */
 	array = (ArrayType *) PG_DETOAST_DATUM(datum);
 
+#ifdef DEBUG
+	elog(NOTICE, "LWGEOM_makeline_garray: array detoasted");
+#endif
+
 	/* Get number of geometries in array */
 	nelems = ArrayGetNItems(ARR_NDIM(array), ARR_DIMS(array));
 
+#ifdef DEBUG
+	elog(NOTICE, "LWGEOM_makeline_garray: array has %d elements", nelems);
+#endif
+
 	/* Return null on 0-elements input array */
 	if ( nelems == 0 )
 	{
@@ -1798,6 +1828,10 @@ Datum LWGEOM_makeline_garray(PG_FUNCTION_ARGS)
 		if ( TYPE_GETTYPE(geoms[i]->type) != POINTTYPE ) continue;
 		lwpoints[npoints++] =
 			lwpoint_deserialize(SERIALIZED_FORM(geoms[i]));
+#ifdef DEBUG
+		elog(NOTICE, "LWGEOM_makeline_garray: element %d deserialized",
+			i);
+#endif
 	}
 
 	/* Return null on 0-points input array */
@@ -1807,6 +1841,10 @@ Datum LWGEOM_makeline_garray(PG_FUNCTION_ARGS)
 		PG_RETURN_NULL();
 	}
 
+#ifdef DEBUG
+	elog(NOTICE, "LWGEOM_makeline_garray: point elements: %d", npoints);
+#endif
+
 	outlwg = (LWGEOM *)make_lwline(-1, npoints, lwpoints);
 
 	size = lwgeom_serialize_size(outlwg);
-- 
2.40.0