postgres.git / summary / log / commit / refs

commit    6c0be49b2ab7a673656d385ba09fc0c1657c6e33
Author:   Peter Eisentraut <peter_e@gmx.net>
Date:     Sun Jul 17 13:37:33 2016 +0000

    Use correct symbol for minimum int64 value
    
    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 writing out the constant
    the long way, as it is done elsewhere in the code.


 contrib/btree_gin/btree_gin.c | 10 ++--------
 1 file changed, 2 insertions(+), 8 deletions(-)

diff --git a/contrib/btree_gin/btree_gin.c b/contrib/btree_gin/btree_gin.c
index 5e6003f63cb..1cff0e3d386 100644
--- a/contrib/btree_gin/btree_gin.c
+++ b/contrib/btree_gin/btree_gin.c
@@ -213,10 +213,7 @@ GIN_SUPPORT(int4)
 static Datum
 leftmostvalue_int8(void)
 {
-	/*
-	 * Use sequence's definition to keep compatibility.
-	 */
-	return Int64GetDatum(SEQ_MINVALUE);
+	return Int64GetDatum(-INT64CONST(0x7FFFFFFFFFFFFFFF) - 1);
 }
 static TypeInfo TypeInfo_int8 = {false, leftmostvalue_int8, btint8cmp};
 
@@ -243,10 +240,7 @@ GIN_SUPPORT(float8)
 static Datum
 leftmostvalue_money(void)
 {
-	/*
-	 * Use sequence's definition to keep compatibility.
-	 */
-	return Int64GetDatum(SEQ_MINVALUE);
+	return Int64GetDatum(-INT64CONST(0x7FFFFFFFFFFFFFFF) - 1);
 }
 static TypeInfo TypeInfo_money = {false, leftmostvalue_money, cash_cmp};
 


[parent: 89b301104a6c]