rfc:remove_preg_replace_eval_modifier

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:remove_preg_replace_eval_modifier [2012/02/05 15:13] nikicrfc:remove_preg_replace_eval_modifier [2012/03/13 17:31] – Just remove the vote, as I broke it anyways nikic
Line 3: Line 3:
   * Date: 2012-02-04   * Date: 2012-02-04
   * Author: Nikita Popov <nikic@php.net>   * Author: Nikita Popov <nikic@php.net>
-  * Status: In Draft+  * Status: Implemented
  
 ===== Summary ===== ===== Summary =====
Line 61: Line 61:
 === But we don't remove eval() either, do we? === === But we don't remove eval() either, do we? ===
  
-The use of the ''/e'' modifier is not comparable to the use of ''eval()''//Any// use of ''/e'' can be replaced +The use of the ''/e'' modifier is not comparable to the use of ''eval()''*Anyuse of ''/e'' can be replaced 
-with a safer and cleaner use of ''preg_replace_callback''. ''eval()'' on the other hand has //some// valid use cases+with a safer and cleaner use of ''preg_replace_callback''. ''eval()'' on the other hand has *somevalid use cases
 which can not be implemented in any other (sane) way. which can not be implemented in any other (sane) way.
  
Line 102: Line 102:
 replaced by a callback there would be no loss in functionality. replaced by a callback there would be no loss in functionality.
  
-The time line for deprecation and removal is subject to discussion.+===== Vote ===== 
 + 
 +The vote ended with 23 in favor and 4 against the proposal. 
 + 
 +===== Current state ===== 
 + 
 +The ''/e'' modifier has been deprecated in trunk in http://svn.php.net/viewvc?view=revision&revision=323862. It will 
 +be removed at some later point in time.
rfc/remove_preg_replace_eval_modifier.txt · Last modified: 2017/09/22 13:28 by 127.0.0.1