dsp                                      Wed, 25 Nov 2009 18:15:25 +0000

Revision: http://svn.php.net/viewvc?view=revision&revision=291320

Log:
Really use 62 as the upper boundary as requested and not 32.

Changed paths:
    U   php/php-src/branches/PHP_5_2/ext/gmp/gmp.c
    U   php/php-src/branches/PHP_5_2/ext/gmp/tests/bug50283.phpt
    U   php/php-src/branches/PHP_5_3/ext/gmp/gmp.c
    U   php/php-src/branches/PHP_5_3/ext/gmp/tests/bug50283.phpt
    U   php/php-src/trunk/ext/gmp/gmp.c
    U   php/php-src/trunk/ext/gmp/tests/bug50283.phpt

Modified: php/php-src/branches/PHP_5_2/ext/gmp/gmp.c
===================================================================
--- php/php-src/branches/PHP_5_2/ext/gmp/gmp.c	2009-11-25 16:53:25 UTC (rev 291319)
+++ php/php-src/branches/PHP_5_2/ext/gmp/gmp.c	2009-11-25 18:15:25 UTC (rev 291320)
@@ -817,7 +817,7 @@
 			break;
 	}

-	if ((base < 2 && base > -2) || base > 36 || base < -36) {
+	if ((base < 2 && base > -2) || base > 62 || base < -36) {
 		php_error_docref(NULL TSRMLS_CC, E_WARNING, "Bad base for conversion: %d", base);
 		RETURN_FALSE;
 	}

Modified: php/php-src/branches/PHP_5_2/ext/gmp/tests/bug50283.phpt
===================================================================
--- php/php-src/branches/PHP_5_2/ext/gmp/tests/bug50283.phpt	2009-11-25 16:53:25 UTC (rev 291319)
+++ php/php-src/branches/PHP_5_2/ext/gmp/tests/bug50283.phpt	2009-11-25 18:15:25 UTC (rev 291320)
@@ -5,25 +5,29 @@
 --FILE--
 <?php
 $a = gmp_init("0x41682179fbf5");
-printf("Decimal: %s, 36-based: %s\n", gmp_strval($a), gmp_strval($a,-36));
+printf("Decimal: %s, -36-based: %s\n", gmp_strval($a), gmp_strval($a,-36));
 printf("Decimal: %s, 36-based: %s\n", gmp_strval($a), gmp_strval($a,36));
-printf("Decimal: %s, 36-based: %s\n", gmp_strval($a), gmp_strval($a,-1));
-printf("Decimal: %s, 36-based: %s\n", gmp_strval($a), gmp_strval($a,1));
-printf("Decimal: %s, 36-based: %s\n", gmp_strval($a), gmp_strval($a,-37));
-printf("Decimal: %s, 36-based: %s\n", gmp_strval($a), gmp_strval($a,37));
+printf("Decimal: %s, -1-based: %s\n", gmp_strval($a), gmp_strval($a,-1));
+printf("Decimal: %s, 1-based: %s\n", gmp_strval($a), gmp_strval($a,1));
+printf("Decimal: %s, -37-based: %s\n", gmp_strval($a), gmp_strval($a,-37));
+printf("Decimal: %s, 37-based: %s\n", gmp_strval($a), gmp_strval($a,37));
+printf("Decimal: %s, 62-based: %s\n", gmp_strval($a), gmp_strval($a,62));
+printf("Decimal: %s, 63-based: %s\n", gmp_strval($a), gmp_strval($a,63));
 ?>
 --EXPECTF--
-Decimal: 71915494046709, 36-based: PHPISCOOL
+Decimal: 71915494046709, -36-based: PHPISCOOL
 Decimal: 71915494046709, 36-based: phpiscool

 Warning: gmp_strval(): Bad base for conversion: -1 in %s on line 5
-Decimal: 71915494046709, 36-based:
+Decimal: 71915494046709, -1-based:

 Warning: gmp_strval(): Bad base for conversion: 1 in %s on line 6
-Decimal: 71915494046709, 36-based:
+Decimal: 71915494046709, 1-based:

 Warning: gmp_strval(): Bad base for conversion: -37 in %s on line 7
-Decimal: 71915494046709, 36-based:
+Decimal: 71915494046709, -37-based:
+Decimal: 71915494046709, 37-based: KHKATELJF
+Decimal: 71915494046709, 62-based: KQ6yq741

-Warning: gmp_strval(): Bad base for conversion: 37 in %s on line 8
-Decimal: 71915494046709, 36-based:
+Warning: gmp_strval(): Bad base for conversion: 63 in %s on line 10
+Decimal: 71915494046709, 63-based:

Modified: php/php-src/branches/PHP_5_3/ext/gmp/gmp.c
===================================================================
--- php/php-src/branches/PHP_5_3/ext/gmp/gmp.c	2009-11-25 16:53:25 UTC (rev 291319)
+++ php/php-src/branches/PHP_5_3/ext/gmp/gmp.c	2009-11-25 18:15:25 UTC (rev 291320)
@@ -796,7 +796,7 @@
 		return;
 	}

-	if ((base < 2 && base > -2) || base > 36 || base < -36) {
+	if ((base < 2 && base > -2) || base > 62 || base < -36) {
 		php_error_docref(NULL TSRMLS_CC, E_WARNING, "Bad base for conversion: %ld", base);
 		RETURN_FALSE;
 	}

Modified: php/php-src/branches/PHP_5_3/ext/gmp/tests/bug50283.phpt
===================================================================
--- php/php-src/branches/PHP_5_3/ext/gmp/tests/bug50283.phpt	2009-11-25 16:53:25 UTC (rev 291319)
+++ php/php-src/branches/PHP_5_3/ext/gmp/tests/bug50283.phpt	2009-11-25 18:15:25 UTC (rev 291320)
@@ -5,25 +5,29 @@
 --FILE--
 <?php
 $a = gmp_init("0x41682179fbf5");
-printf("Decimal: %s, 36-based: %s\n", gmp_strval($a), gmp_strval($a,-36));
+printf("Decimal: %s, -36-based: %s\n", gmp_strval($a), gmp_strval($a,-36));
 printf("Decimal: %s, 36-based: %s\n", gmp_strval($a), gmp_strval($a,36));
-printf("Decimal: %s, 36-based: %s\n", gmp_strval($a), gmp_strval($a,-1));
-printf("Decimal: %s, 36-based: %s\n", gmp_strval($a), gmp_strval($a,1));
-printf("Decimal: %s, 36-based: %s\n", gmp_strval($a), gmp_strval($a,-37));
-printf("Decimal: %s, 36-based: %s\n", gmp_strval($a), gmp_strval($a,37));
+printf("Decimal: %s, -1-based: %s\n", gmp_strval($a), gmp_strval($a,-1));
+printf("Decimal: %s, 1-based: %s\n", gmp_strval($a), gmp_strval($a,1));
+printf("Decimal: %s, -37-based: %s\n", gmp_strval($a), gmp_strval($a,-37));
+printf("Decimal: %s, 37-based: %s\n", gmp_strval($a), gmp_strval($a,37));
+printf("Decimal: %s, 62-based: %s\n", gmp_strval($a), gmp_strval($a,62));
+printf("Decimal: %s, 63-based: %s\n", gmp_strval($a), gmp_strval($a,63));
 ?>
 --EXPECTF--
-Decimal: 71915494046709, 36-based: PHPISCOOL
+Decimal: 71915494046709, -36-based: PHPISCOOL
 Decimal: 71915494046709, 36-based: phpiscool

 Warning: gmp_strval(): Bad base for conversion: -1 in %s on line 5
-Decimal: 71915494046709, 36-based:
+Decimal: 71915494046709, -1-based:

 Warning: gmp_strval(): Bad base for conversion: 1 in %s on line 6
-Decimal: 71915494046709, 36-based:
+Decimal: 71915494046709, 1-based:

 Warning: gmp_strval(): Bad base for conversion: -37 in %s on line 7
-Decimal: 71915494046709, 36-based:
+Decimal: 71915494046709, -37-based:
+Decimal: 71915494046709, 37-based: KHKATELJF
+Decimal: 71915494046709, 62-based: KQ6yq741

-Warning: gmp_strval(): Bad base for conversion: 37 in %s on line 8
-Decimal: 71915494046709, 36-based:
+Warning: gmp_strval(): Bad base for conversion: 63 in %s on line 10
+Decimal: 71915494046709, 63-based:

Modified: php/php-src/trunk/ext/gmp/gmp.c
===================================================================
--- php/php-src/trunk/ext/gmp/gmp.c	2009-11-25 16:53:25 UTC (rev 291319)
+++ php/php-src/trunk/ext/gmp/gmp.c	2009-11-25 18:15:25 UTC (rev 291320)
@@ -804,7 +804,7 @@
 		return;
 	}

-	if ((base < 2 && base > -2) || base > 36 || base < -36) {
+	if ((base < 2 && base > -2) || base > 62 || base < -36) {
 		php_error_docref(NULL TSRMLS_CC, E_WARNING, "Bad base for conversion: %ld", base);
 		RETURN_FALSE;
 	}

Modified: php/php-src/trunk/ext/gmp/tests/bug50283.phpt
===================================================================
--- php/php-src/trunk/ext/gmp/tests/bug50283.phpt	2009-11-25 16:53:25 UTC (rev 291319)
+++ php/php-src/trunk/ext/gmp/tests/bug50283.phpt	2009-11-25 18:15:25 UTC (rev 291320)
@@ -5,25 +5,29 @@
 --FILE--
 <?php
 $a = gmp_init("0x41682179fbf5");
-printf("Decimal: %s, 36-based: %s\n", gmp_strval($a), gmp_strval($a,-36));
+printf("Decimal: %s, -36-based: %s\n", gmp_strval($a), gmp_strval($a,-36));
 printf("Decimal: %s, 36-based: %s\n", gmp_strval($a), gmp_strval($a,36));
-printf("Decimal: %s, 36-based: %s\n", gmp_strval($a), gmp_strval($a,-1));
-printf("Decimal: %s, 36-based: %s\n", gmp_strval($a), gmp_strval($a,1));
-printf("Decimal: %s, 36-based: %s\n", gmp_strval($a), gmp_strval($a,-37));
-printf("Decimal: %s, 36-based: %s\n", gmp_strval($a), gmp_strval($a,37));
+printf("Decimal: %s, -1-based: %s\n", gmp_strval($a), gmp_strval($a,-1));
+printf("Decimal: %s, 1-based: %s\n", gmp_strval($a), gmp_strval($a,1));
+printf("Decimal: %s, -37-based: %s\n", gmp_strval($a), gmp_strval($a,-37));
+printf("Decimal: %s, 37-based: %s\n", gmp_strval($a), gmp_strval($a,37));
+printf("Decimal: %s, 62-based: %s\n", gmp_strval($a), gmp_strval($a,62));
+printf("Decimal: %s, 63-based: %s\n", gmp_strval($a), gmp_strval($a,63));
 ?>
 --EXPECTF--
-Decimal: 71915494046709, 36-based: PHPISCOOL
+Decimal: 71915494046709, -36-based: PHPISCOOL
 Decimal: 71915494046709, 36-based: phpiscool

 Warning: gmp_strval(): Bad base for conversion: -1 in %s on line 5
-Decimal: 71915494046709, 36-based:
+Decimal: 71915494046709, -1-based:

 Warning: gmp_strval(): Bad base for conversion: 1 in %s on line 6
-Decimal: 71915494046709, 36-based:
+Decimal: 71915494046709, 1-based:

 Warning: gmp_strval(): Bad base for conversion: -37 in %s on line 7
-Decimal: 71915494046709, 36-based:
+Decimal: 71915494046709, -37-based:
+Decimal: 71915494046709, 37-based: KHKATELJF
+Decimal: 71915494046709, 62-based: KQ6yq741

-Warning: gmp_strval(): Bad base for conversion: 37 in %s on line 8
-Decimal: 71915494046709, 36-based:
+Warning: gmp_strval(): Bad base for conversion: 63 in %s on line 10
+Decimal: 71915494046709, 63-based:
-- 
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php

Reply via email to