Remove wrongly backpatched piece of code in cube.c

feodor · feodor · commit 29ab1567e7ea · 2018-04-10T14:58:46.000+03:00
Due to sloppy division of changes between f50c80d (which was not back-patched) and 563a053, this piece of code was wrongly backpatched to REL_10_STABLE and REL9_6_STABLE. This code never causes real error because its condition is never satisfied, but it's a dead code, which needs to be removed. Alexander Korotkov per gripe from Tom Lane
diff --git a/contrib/cube/cube.c b/contrib/cube/cube.c
@@ -1555,7 +1555,6 @@ cube_coord_llur(PG_FUNCTION_ARGS)
 {
 	NDBOX	   *cube = PG_GETARG_NDBOX(0);
 	int			coord = PG_GETARG_INT32(1);
-	bool		inverse = false;
 	float8		result;
 
 	/* 0 is the only unsupported coordinate value */
@@ -1593,10 +1592,6 @@ cube_coord_llur(PG_FUNCTION_ARGS)
 		result = 0.0;
 	}
 
-	/* Inverse value if needed */
-	if (inverse)
-		result = -result;
-
 	PG_RETURN_FLOAT8(result);
 }
 

Original file line number	Diff line number	Diff line change
`@@ -1555,7 +1555,6 @@ cube_coord_llur(PG_FUNCTION_ARGS)`
`1555`	`1555`	`{`
`1556`	`1556`	`NDBOX *cube = PG_GETARG_NDBOX(0);`
`1557`	`1557`	`int coord = PG_GETARG_INT32(1);`
`1558`		`- bool inverse = false;`
`1559`	`1558`	`float8 result;`
`1560`	`1559`
`1561`	`1560`	`/* 0 is the only unsupported coordinate value */`
`@@ -1593,10 +1592,6 @@ cube_coord_llur(PG_FUNCTION_ARGS)`
`1593`	`1592`	`result = 0.0;`
`1594`	`1593`	`}`
`1595`	`1594`
`1596`		`- /* Inverse value if needed */`
`1597`		`- if (inverse)`
`1598`		`- result = -result;`
`1599`		`-`
`1600`	`1595`	`PG_RETURN_FLOAT8(result);`
`1601`	`1596`	`}`
`1602`	`1597`