summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorGabriel Caruso <carusogabriel34@gmail.com>2019-05-08 16:42:02 -0300
committerGabriel Caruso <carusogabriel34@gmail.com>2019-05-18 15:59:07 -0300
commit714d9fc358640069bda5540c2b1136a6241c4c94 (patch)
treedf6fa4174cc9c354dce99dd90009f5c4987a0ef6
parent256100a9b57113915c4ea7f203a9d094eaa129a0 (diff)
downloadphp-git-714d9fc358640069bda5540c2b1136a6241c4c94.tar.gz
Make chr ZPP failure message consistent with ext/standard
-rw-r--r--ext/standard/string.c6
-rw-r--r--ext/standard/tests/array/array_filter_variation9.phpt2
-rw-r--r--ext/standard/tests/strings/chr_error.phptbin783 -> 823 bytes
-rw-r--r--ext/standard/tests/strings/chr_ord.phptbin3171 -> 0 bytes
-rw-r--r--ext/standard/tests/strings/chr_variation1.phpt29
5 files changed, 7 insertions, 30 deletions
diff --git a/ext/standard/string.c b/ext/standard/string.c
index 446de00908..e0141ffaea 100644
--- a/ext/standard/string.c
+++ b/ext/standard/string.c
@@ -2792,11 +2792,7 @@ PHP_FUNCTION(chr)
{
zend_long c;
- if (ZEND_NUM_ARGS() != 1) {
- WRONG_PARAM_COUNT;
- }
-
- ZEND_PARSE_PARAMETERS_START_EX(ZEND_PARSE_PARAMS_QUIET, 1, 1)
+ ZEND_PARSE_PARAMETERS_START(1, 1)
Z_PARAM_LONG(c)
ZEND_PARSE_PARAMETERS_END_EX(c = 0);
diff --git a/ext/standard/tests/array/array_filter_variation9.phpt b/ext/standard/tests/array/array_filter_variation9.phpt
index ed89d13d35..a6f5789eae 100644
--- a/ext/standard/tests/array/array_filter_variation9.phpt
+++ b/ext/standard/tests/array/array_filter_variation9.phpt
@@ -45,6 +45,8 @@ array(6) {
[5]=>
int(1000)
}
+
+Warning: chr() expects parameter 1 to be int, string given in %s on line %d
array(8) {
[0]=>
int(0)
diff --git a/ext/standard/tests/strings/chr_error.phpt b/ext/standard/tests/strings/chr_error.phpt
index d5fb974559..4d595543bb 100644
--- a/ext/standard/tests/strings/chr_error.phpt
+++ b/ext/standard/tests/strings/chr_error.phpt
Binary files differ
diff --git a/ext/standard/tests/strings/chr_ord.phpt b/ext/standard/tests/strings/chr_ord.phpt
deleted file mode 100644
index be0b1a32da..0000000000
--- a/ext/standard/tests/strings/chr_ord.phpt
+++ /dev/null
Binary files differ
diff --git a/ext/standard/tests/strings/chr_variation1.phpt b/ext/standard/tests/strings/chr_variation1.phpt
index 85a2219ecb..c962aa0e78 100644
--- a/ext/standard/tests/strings/chr_variation1.phpt
+++ b/ext/standard/tests/strings/chr_variation1.phpt
@@ -38,11 +38,6 @@ $inputs = array (
-20.5,
1.1234e6,
- // array values
-/*8*/ array(),
- array(0),
- array(1, 2),
-
// boolean values
/*11*/ true,
false,
@@ -53,12 +48,6 @@ $inputs = array (
/*15*/ NULL,
null,
- // objects
-/*17*/ new sample(),
-
- // resource
-/*18*/ $file_handle,
-
// undefined variable
/*19*/ @$undefined_var,
@@ -95,29 +84,19 @@ string(2) "ec"
-- Iteration 7 --
string(2) "48"
-- Iteration 8 --
-string(2) "00"
+string(2) "01"
-- Iteration 9 --
string(2) "00"
-- Iteration 10 --
-string(2) "00"
--- Iteration 11 --
string(2) "01"
+-- Iteration 11 --
+string(2) "00"
-- Iteration 12 --
string(2) "00"
-- Iteration 13 --
-string(2) "01"
+string(2) "00"
-- Iteration 14 --
string(2) "00"
-- Iteration 15 --
string(2) "00"
--- Iteration 16 --
-string(2) "00"
--- Iteration 17 --
-string(2) "00"
--- Iteration 18 --
-string(2) "00"
--- Iteration 19 --
-string(2) "00"
--- Iteration 20 --
-string(2) "00"
===DONE===