rfc:concatenation_precedence

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
rfc:concatenation_precedence [2019/03/28 11:00] – PHP 8 only bwoebirfc:concatenation_precedence [2020/08/01 23:54] (current) – RFC was implemented carusogabriel
Line 1: Line 1:
 ====== PHP RFC: Change the precedence of the concatenation operator ====== ====== PHP RFC: Change the precedence of the concatenation operator ======
-  * Version: 0.9+  * Version: 1.0
   * Date: 2019-03-28   * Date: 2019-03-28
   * Author: Bob Weinand, bobwei9@hotmail.com   * Author: Bob Weinand, bobwei9@hotmail.com
-  * Status: Draft+  * Status: Implemented
   * First Published at: http://wiki.php.net/rfc/concatenation_precedence   * First Published at: http://wiki.php.net/rfc/concatenation_precedence
  
Line 28: Line 28:
 This is counter-intuitive though: you rarely want to add or subtract concatenated strings which in general are not numbers. However, given PHPs capability of seamlessly converting an integer to a string, concatenation of these values is desired. This is counter-intuitive though: you rarely want to add or subtract concatenated strings which in general are not numbers. However, given PHPs capability of seamlessly converting an integer to a string, concatenation of these values is desired.
  
-Thus, the RFC proposes to give '.' an inferior precedence to '+' and '-', so that additions and subtractions are always performed before the concatenation.+Thus, the RFC proposes to give '.' an inferior precedence to '+' and '-', so that additions and subtractions are always performed before the concatenation. Concretely, the new precedence will be right below '<<' and '>>' operators, given them also being "math" operators and not usable with non-numeric strings.
  
 ===== Backward Incompatible Changes ===== ===== Backward Incompatible Changes =====
  
-Every unparenthesized expression featuring an '-' or '+' after a '.' will change behavior. As an example, the expression <php>"3" . "5" + 7</php> will now be equal to 312 instead of previously 42.+Every unparenthesized expression featuring an '-' or '+' after a '.' will change behavior. As an example, the expression <php>"3" . "5" + 7</php> will now be equal to "312" instead of previously 42.
  
 While this is a subtle behavior change in that it will give different outputs without notice or warning, it is trivially possible to statically analyze the code and find all instances where this happens. As to my knowledge these occurrences are quite rare as it almost always is an error in the current form, rendering the impact minimal. While this is a subtle behavior change in that it will give different outputs without notice or warning, it is trivially possible to statically analyze the code and find all instances where this happens. As to my knowledge these occurrences are quite rare as it almost always is an error in the current form, rendering the impact minimal.
 +
 +As Nikita mentioned on the mailing list: http://news.php.net/php.internals/105442 - the impact to existing open-source code is de facto invisible; all found occurrences are actual bugs. This is a strong indicator that the overall impact will also be very minimal.
  
 ===== Proposed PHP Version(s) ===== ===== Proposed PHP Version(s) =====
Line 40: Line 42:
 PHP 8, with a deprecation notice in PHP 7.4 upon encountering an unparenthesized expression containing an '.' before a '+' or '-'.  PHP 8, with a deprecation notice in PHP 7.4 upon encountering an unparenthesized expression containing an '.' before a '+' or '-'. 
  
-===== Proposed Voting Choices =====+===== Vote ===== 
 + 
 +Voting started 2019-04-30 and ends 2019-05-14. The first (primary) voting requires a 2/3 majority. The second (secondary) voting requires a 50%+1 majority. 
 + 
 +<doodle title="Change the precedence of the concatenation operator as proposed in PHP 8?" auth="bwoebi" voteType="single" closed="true"> 
 +   * Yes 
 +   * No 
 +</doodle>
  
-"Change the precedence in PHP 8", "No"+<doodle title="Emit a deprecation notice in PHP 7.4 if the RFC is accepted?" auth="bwoebi" voteType="single" closed="true"> 
 +   * Yes 
 +   * No 
 +</doodle>
  
 ===== Patch ===== ===== Patch =====
  
-The patch for the change itself is trivial, requiring small changes in the zend_ast.c and zend_language_parser.y to change the precendences. The emission of deprecation is a bit more contrived... I'll add it later.+  * https://github.com/php/php-src/pull/4001 (PHP 7.4) 
 +  * https://github.com/php/php-src/pull/4002 (PHP 8)
rfc/concatenation_precedence.1553770831.txt.gz · Last modified: 2019/03/28 11:00 by bwoebi

Page Tools