rfc:deprecate_mb_ereg_replace_eval_option

Differences

This shows you the differences between two versions of the page.

Link to this comparison view

Both sides previous revisionPrevious revision
Next revision
Previous revision
Last revisionBoth sides next revision
rfc:deprecate_mb_ereg_replace_eval_option [2016/01/04 16:42] realitykingrfc:deprecate_mb_ereg_replace_eval_option [2016/07/11 12:10] – this RFC has been implemented cmb
Line 1: Line 1:
-====== PHP RFC: Deprecate nb_ereg_replace eval option ====== +====== PHP RFC: Deprecate mb_ereg_replace eval option ====== 
-  * Version: 0.1+  * Version: 1.0
   * Date: 2016-01-04   * Date: 2016-01-04
   * Author: Rouven Weßling, me@rouvenwessling.de   * Author: Rouven Weßling, me@rouvenwessling.de
-  * Status: Under Discussion+  * Status: Implemented (PHP 7.1)
   * First Published at: https://wiki.php.net/rfc/deprecate_mb_ereg_replace_eval_option   * First Published at: https://wiki.php.net/rfc/deprecate_mb_ereg_replace_eval_option
  
Line 20: Line 20:
 ===== Proposal ===== ===== Proposal =====
  
-Emit an E_DEPRECATED error whenever ''mb_ereg_replace_callback'' or ''mb_eregi_replace_callback'' is called with the ''e'' option.+Emit an E_DEPRECATED error whenever ''mb_ereg_replace'' or ''mb_eregi_replace'' is called with the ''e'' option.
  
 ===== Backward Incompatible Changes ===== ===== Backward Incompatible Changes =====
Line 35: Line 35:
  
 ===== Proposed Voting Choices ===== ===== Proposed Voting Choices =====
 +
 +Should an E_DEPRECATED error be emitted whenever mb_ereg_replace or mb_eregi_replace is called with the e option?
  
 This vote will require a 50%+1 majority. This vote will require a 50%+1 majority.
 +
 +Voting opened on January 28th 2016, to close on February 4th.
 +
 +<doodle title="Deprecate mb_ereg_replace eval option" auth="realityking" voteType="single" closed="true">
 +   * Yes
 +   * No
 +</doodle>
  
 ===== Patches and Tests ===== ===== Patches and Tests =====
  
-Patch is not yet written, but it's quite trivial so I'll do it myself.+A patch is available: <https://github.com/php/php-src/pull/1994> (merged).
  
 ===== References ===== ===== References =====
  
- * https://wiki.php.net/rfc/remove_preg_replace_eval_modifier +[[rfc:remove_preg_replace_eval_modifier| RFC: Remove preg_replace /e modifier]]
rfc/deprecate_mb_ereg_replace_eval_option.txt · Last modified: 2017/09/22 13:28 by 127.0.0.1