]> granicus.if.org Git - php/commitdiff
Merge branch 'PHP-7.4'
authorGeorge Peter Banyard <girgias@php.net>
Tue, 7 Apr 2020 20:49:17 +0000 (22:49 +0200)
committerGeorge Peter Banyard <girgias@php.net>
Tue, 7 Apr 2020 20:49:17 +0000 (22:49 +0200)
* PHP-7.4:
  Revert "Went to fast and forgot to update tests"
  Revert "Fix Bug #79448 0 is a valid Unicode codepoint, but mb_substitute_character(0) fails"

1  2 
ext/mbstring/mbstring.c
ext/mbstring/tests/mb_substitute_character.phpt
ext/mbstring/tests/mb_substitute_character_basic.phpt
ext/mbstring/tests/mb_substitute_character_variation1.phpt

Simple merge
index 490542837f5db45e989159c42b5a9537714ac10a,6bc533f0386e89f5d630ae009d0ef601a32492aa..9e50823ef42b503107818bfc98b8b99e4d5e6746
@@@ -25,7 -25,7 +25,7 @@@ var_dump(bin2hex(mb_convert_encoding("\
  
  var_dump(mb_substitute_character('BAD_NAME'));
  ?>
----EXPECT--
++--EXPECTF--
  bool(true)
  int(12356)
  string(8) "82a282a0"
@@@ -38,4 -38,5 +38,6 @@@ string(4) "82a0
  bool(true)
  string(6) "entity"
  string(20) "262378323636303b82a0"
- bool(true)
 -ERR: Warning
++
++Warning: mb_substitute_character(): Unknown character in %s on line %d
+ bool(false)
index 39af2d0c76ce7f42a6db22ae7fed720493a5c8fe,9fa3a5b1acad1cefb201d2cadd20e8ada32d241b..63ae10f76c1ef21a859604939d25d365a0991ef2
@@@ -27,7 -27,8 +27,7 @@@ var_dump( mb_substitute_character() )
  var_dump( mb_substitute_character("b") );
  
  ?>
- --EXPECT--
 -===DONE===
+ --EXPECTF--
  *** Testing mb_substitute_character() : basic functionality ***
  int(63)
  bool(true)
@@@ -36,4 -37,7 +36,6 @@@ bool(true
  int(1234)
  bool(true)
  string(4) "none"
- bool(true)
+ Warning: mb_substitute_character(): Unknown character in %s on line %d
+ bool(false)
 -===DONE===
index 819d60493ca666430e864233671df96dda6abefa,f738876469fcd06dbf9d96c40db0c6a7e28c7845..53a21c87878f966508ac00ff5ae371c70c3a9f95
@@@ -209,7 -221,10 +220,9 @@@ Error: 8 - Object of class classWithout
  bool(true)
  
  --undefined var--
- bool(true)
+ Error: 2 - mb_substitute_character(): Unknown character, %s(%d)
+ bool(false)
  
  --unset var--
- bool(true)
+ Error: 2 - mb_substitute_character(): Unknown character, %s(%d)
+ bool(false)
 -===DONE===