Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:88847
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.160.172 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <CALYT1YpZVz-ixf-v9GdCM3EcYOdt9zAOH03_2rA3BK4FRDuD6w@mail.gmail.com>
References: <CALYT1YpZVz-ixf-v9GdCM3EcYOdt9zAOH03_2rA3BK4FRDuD6w@mail.gmail.com>
Date: Thu, 15 Oct 2015 22:28:38 -0300
Message-ID: <CAOsHV+uF_cqBfYbo3BXK28s_2Mm5DDb0Rwf2pubyfpc_3fhJjw@mail.gmail.com>
To: Sammy Kaye Powers <me@sammyk.me>
Cc: PHP Internals <internals@lists.php.net>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable
Subject: Re: [PHP-DEV] [RFC][DISCUSSION] Revisit trailing commas in function arguments
From: marcio.web2@gmail.com (Marcio Almada)

Hi,

2015-10-14 16:25 GMT-03:00 Sammy Kaye Powers <me@sammyk.me>:
> Hello internals friends!
>
> I'd like to open a discussion on the RFC to allow trailing commas in
> function arguments.
>
> https://wiki.php.net/rfc/revisit-trailing-comma-function-args
>
> Discuss! :)
>
> Thanks,
> Sammy Kaye Powers
> sammyk.me

Sammy, I think this proposal is too narrow. If we are going to bother
to change the language, then let's do it consistently and allow
trailing commas on all lists in the grammar. These are the ones on
mind right now:

- use declarations
- group use declarations
- function call argument list
- class member lists (both constants and properties)
- argument list declarations
- arrays (already allowed)

As exemplified in this gist:
https://gist.github.com/marcioAlmada/75f8f1d47da5dcac2e57

Why? Because it prevents a reality where each PHP minor version
introduces trailing commas in a new different place, hurting code
portability. Because it couldn't get more consistent. And more
importantly: because we would get rid of this discussion about
trailing commas, forever j/k :)

ty,
M=C3=A1rcio