Normalize gmp_cmp() to -1/0/+1

Girgias · Girgias · commit cf10074433d1 · 2024-11-16T16:34:53.000Z
diff --git a/ext/gmp/gmp.c b/ext/gmp/gmp.c
@@ -474,7 +474,8 @@ static int gmp_compare(zval *op1, zval *op2) /* {{{ */
 		}
 		return ZEND_UNCOMPARABLE;
 	}
-	return mpz_cmp(gmp_op1, gmp_op2);
+
+	return ZEND_THREEWAY_COMPARE(mpz_cmp(gmp_op1, gmp_op2), 0);
 }
 /* }}} */
 
@@ -1429,7 +1430,7 @@ ZEND_FUNCTION(gmp_cmp)
 		GMP_Z_PARAM_INTO_MPZ_PTR(gmpnum_b)
 	ZEND_PARSE_PARAMETERS_END();
 
-	RETURN_LONG(mpz_cmp(gmpnum_a, gmpnum_b));
+	RETURN_LONG(ZEND_THREEWAY_COMPARE(mpz_cmp(gmpnum_a, gmpnum_b), 0));
 }
 /* }}} */
 

Original file line number	Diff line number	Diff line change
`@@ -474,7 +474,8 @@ static int gmp_compare(zval op1, zval op2) /* {{{ */`
`474`	`474`	`}`
`475`	`475`	`return ZEND_UNCOMPARABLE;`
`476`	`476`	`}`
`477`		`- return mpz_cmp(gmp_op1, gmp_op2);`
	`477`	`+`
	`478`	`+ return ZEND_THREEWAY_COMPARE(mpz_cmp(gmp_op1, gmp_op2), 0);`
`478`	`479`	`}`
`479`	`480`	`/* }}} */`
`480`	`481`
`@@ -1429,7 +1430,7 @@ ZEND_FUNCTION(gmp_cmp)`
`1429`	`1430`	`GMP_Z_PARAM_INTO_MPZ_PTR(gmpnum_b)`
`1430`	`1431`	`ZEND_PARSE_PARAMETERS_END();`
`1431`	`1432`
`1432`		`- RETURN_LONG(mpz_cmp(gmpnum_a, gmpnum_b));`
	`1433`	`+ RETURN_LONG(ZEND_THREEWAY_COMPARE(mpz_cmp(gmpnum_a, gmpnum_b), 0));`
`1433`	`1434`	`}`
`1434`	`1435`	`/* }}} */`
`1435`	`1436`