Fix compiler warning in rangetypes_spgist.c.

tglsfdc · tglsfdc · commit a96e22528335 · 2017-11-18T16:46:30.000-05:00
On gcc 7.2.0, comparing pointer to (Datum) 0 produces a warning. Treat it as a simple pointer to avoid that; this is more consistent with comparable code elsewhere, anyway. Tomas Vondra Discussion: https://postgr.es/m/99410021-61ef-9a9a-9bc8-f733ece637ee@2ndquadrant.com
diff --git a/src/backend/utils/adt/rangetypes_spgist.c b/src/backend/utils/adt/rangetypes_spgist.c
@@ -563,7 +563,7 @@ spg_range_quad_inner_consistent(PG_FUNCTION_ARGS)
 					 * for lower or upper bounds to be adjacent. Deserialize
 					 * previous centroid range if present for checking this.
 					 */
-					if (in->traversalValue != (Datum) 0)
+					if (in->traversalValue)
 					{
 						prevCentroid = DatumGetRangeType(in->traversalValue);
 						range_deserialize(typcache, prevCentroid,

Original file line number	Diff line number	Diff line change
`@@ -563,7 +563,7 @@ spg_range_quad_inner_consistent(PG_FUNCTION_ARGS)`
`563`	`563`	`* for lower or upper bounds to be adjacent. Deserialize`
`564`	`564`	`* previous centroid range if present for checking this.`
`565`	`565`	`*/`
`566`		`- if (in->traversalValue != (Datum) 0)`
	`566`	`+ if (in->traversalValue)`
`567`	`567`	`{`
`568`	`568`	`prevCentroid = DatumGetRangeType(in->traversalValue);`
`569`	`569`	`range_deserialize(typcache, prevCentroid,`