rfc:list-syntax-trailing-commas

Differences

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

Link to this comparison view

Both sides previous revision Previous revision
Next revision
Previous revision
rfc:list-syntax-trailing-commas [2017/01/27 15:22]
sammyk Updated example
rfc:list-syntax-trailing-commas [2017/09/22 13:28] (current)
Line 1: Line 1:
 ====== PHP RFC: Trailing Commas In List Syntax ====== ====== PHP RFC: Trailing Commas In List Syntax ======
   * Version: 0.1   * Version: 0.1
-  * Date: 2015-11-03+  * Date: 2015-11-03 ​(discussion);​ 2017-01-27 (voting)
   * Author: Sammy Kaye Powers, me@sammyk.me   * Author: Sammy Kaye Powers, me@sammyk.me
-  * Status: ​Under Discussion+  * Status: ​Implemented in PHP 7.2 (Grouped namepaces only)
   * First Published at: https://​wiki.php.net/​rfc/​revisit-trailing-comma-function-args,​ https://​wiki.php.net/​rfc/​trailing-comma-function-args   * First Published at: https://​wiki.php.net/​rfc/​revisit-trailing-comma-function-args,​ https://​wiki.php.net/​rfc/​trailing-comma-function-args
  
Line 159: Line 159:
 ===== Proposed PHP Version ===== ===== Proposed PHP Version =====
  
-PHP 7.1+PHP 7.2
  
 ===== Proposed Voting Choices ===== ===== Proposed Voting Choices =====
  
-Each trailing comma list syntax ​would get its own vote and would require ​a 2/3 majority to pass.+Each trailing comma list syntax ​has its own vote and requires ​a 2/3 majority to pass.
  
-  - Function/​method arguments (declarations & calls) + 
-  ​- ​Grouped namepaces +==== Function/​method arguments (declarations & calls) ​==== 
-  ​- ​Interface implementations on a class + 
-  ​- ​Trait implementations on a class +<code php> 
-  ​- ​Class member lists +// Function/​method arguments (call) 
-  ​- ​Inheriting variables from the parent scope in anonymous functions+fooCall( 
 +    $arg1, 
 +    $arg2, 
 +    $arg3, 
 +); 
 +     
 +// Function/​method arguments (declaration) 
 +function something( 
 +    FooBarBazInterface $in, 
 +    FooBarBazInterface $out, 
 +) : bool { 
 +
 +</​code>​ 
 + 
 +<doodle title="​Allow trailing commas in function/​method arguments (declarations and calls)"​ auth="​SammyK"​ voteType="​single"​ closed="​true">​ 
 +   * Yes 
 +   * No 
 +</​doodle>​ 
 + 
 +==== Grouped namepaces ​==== 
 + 
 +<code php> 
 +<?php 
 +use Foo\Bar\{ 
 +    Foo, 
 +    Bar, 
 +    Baz, 
 +}; 
 +</​code>​ 
 + 
 +<doodle title="​Allow trailing commas in grouped namepaces"​ auth="​SammyK"​ voteType="​single"​ closed="​true">​ 
 +   * Yes 
 +   * No 
 +</​doodle>​ 
 + 
 +==== Interface implementations on a class ==== 
 + 
 +<code php> 
 +class Foo implements 
 +    FooInterface,​ 
 +    BarInterface,​ 
 +    BazInterface,​ 
 +
 +
 +</​code>​ 
 + 
 +<doodle title="​Allow trailing commas in interface implementations on a class" auth="​SammyK"​ voteType="​single"​ closed="​true">​ 
 +   * Yes 
 +   * No 
 +</​doodle>​ 
 + 
 +==== Trait implementations on a class ==== 
 + 
 +<code php> 
 +class Foo 
 +
 +    use 
 +        FooTrait, 
 +        BarTrait, 
 +        BazTrait, 
 +    ; 
 +
 +</​code>​ 
 + 
 +<doodle title="​Allow trailing commas in trait implementations on a class" auth="​SammyK"​ voteType="​single"​ closed="​true">​ 
 +   * Yes 
 +   * No 
 +</​doodle>​ 
 + 
 +==== Class member lists ==== 
 + 
 +<code php> 
 +class Foo 
 +
 +    const 
 +        A = 1010, 
 +        B = 1021, 
 +        C = 1032, 
 +        D = 1043, 
 +    ; 
 +    protected 
 +        $a = '​foo',​ 
 +        $b = '​bar',​ 
 +        $c = '​baz',​ 
 +    ; 
 +    private 
 +        $blah, 
 +    ; 
 +
 +</​code>​ 
 + 
 +<doodle title="​Allow trailing commas in class member lists" auth="​SammyK"​ voteType="​single"​ closed="​true">​ 
 +   * Yes 
 +   * No 
 +</​doodle>​ 
 + 
 +==== Inheriting variables from the parent scope in anonymous functions ​==== 
 + 
 +<code php> 
 +$foo = function ($bar) use ( 
 +    $a, 
 +    $b, 
 +    $c, 
 +) { 
 + // . . .  
 +}; 
 +</​code>​ 
 + 
 +<doodle title="​Allow trailing commas in use lists for anonymous functions"​ auth="​SammyK"​ voteType="​single"​ closed="​true">​ 
 +   * Yes 
 +   * No 
 +</​doodle>​
  
 ===== Patches and Tests ===== ===== Patches and Tests =====
  
 The [[https://​github.com/​sgolemon/​php-src/​compare/​master...trailing-comma|original patch by Sara Golemon]]. The [[https://​github.com/​sgolemon/​php-src/​compare/​master...trailing-comma|original patch by Sara Golemon]].
rfc/list-syntax-trailing-commas.1485530569.txt.gz · Last modified: 2017/09/22 13:28 (external edit)