]> granicus.if.org Git - postgresql/commitdiff
Use correct symbol for minimum int64 value
authorPeter Eisentraut <peter_e@gmx.net>
Sun, 17 Jul 2016 13:15:37 +0000 (09:15 -0400)
committerPeter Eisentraut <peter_e@gmx.net>
Sun, 17 Jul 2016 13:15:37 +0000 (09:15 -0400)
The old code used SEQ_MINVALUE to get the smallest int64 value.  This
was done as a convenience to avoid having to deal with INT64_IS_BUSTED,
but that is obsolete now.  Also, it is incorrect because the smallest
int64 value is actually SEQ_MINVALUE-1.  Fix by using PG_INT64_MIN.

contrib/btree_gin/btree_gin.c

index f74e912ed7474ece0a3d2b6e650816ca326c0c3e..030b61097f654c925d751c7e8be3d00427f4a16b 100644 (file)
@@ -223,10 +223,7 @@ GIN_SUPPORT(int4, false, leftmostvalue_int4, btint4cmp)
 static Datum
 leftmostvalue_int8(void)
 {
-       /*
-        * Use sequence's definition to keep compatibility.
-        */
-       return Int64GetDatum(SEQ_MINVALUE);
+       return Int64GetDatum(PG_INT64_MIN);
 }
 
 GIN_SUPPORT(int8, false, leftmostvalue_int8, btint8cmp)
@@ -250,10 +247,7 @@ GIN_SUPPORT(float8, false, leftmostvalue_float8, btfloat8cmp)
 static Datum
 leftmostvalue_money(void)
 {
-       /*
-        * Use sequence's definition to keep compatibility.
-        */
-       return Int64GetDatum(SEQ_MINVALUE);
+       return Int64GetDatum(PG_INT64_MIN);
 }
 
 GIN_SUPPORT(money, false, leftmostvalue_money, cash_cmp)