Use PG_INT32_MIN instead of reiterating the constant.

robertmhaas · robertmhaas · commit cbb2a812d710 · 2016-04-13T07:54:45.000-04:00
Makes no difference, but it's cleaner this way.

Michael Paquier
diff --git a/src/backend/utils/adt/numutils.c b/src/backend/utils/adt/numutils.c
@@ -136,7 +136,7 @@ pg_ltoa(int32 value, char *a)
 	 * Avoid problems with the most negative integer not being representable
 	 * as a positive integer.
 	 */
-	if (value == (-2147483647 - 1))
+	if (value == PG_INT32_MIN)
 	{
 		memcpy(a, "-2147483648", 12);
 		return;

Original file line number	Diff line number	Diff line change
`@@ -136,7 +136,7 @@ pg_ltoa(int32 value, char *a)`
`136`	`136`	`* Avoid problems with the most negative integer not being representable`
`137`	`137`	`* as a positive integer.`
`138`	`138`	`*/`
`139`		`- if (value == (-2147483647 - 1))`
	`139`	`+ if (value == PG_INT32_MIN)`
`140`	`140`	`{`
`141`	`141`	`memcpy(a, "-2147483648", 12);`
`142`	`142`	`return;`