[RFC][VOTE] Trailing commas in all list syntax View as Markdown

Can I apologize already for the imminent failure? :P I think it's very much
ok on parameters and especially arguments. I don't really care about the
others.

2017-01-27 17:05 GMT+01:00 Sammy Kaye Powers me@sammyk.me:

Hello my internals friends!

After a 2 week (ahem... 1 year & 3 month - sorry for dropping the
ball) discussion period, the "Trailing commas in all list syntax" RFC
is now open for voting.

See the discussion here:
internals@lists.php.net/msg81366.html" rel="nofollow" target="_blank">https://www.mail-archive.com/internals@lists.php.net/msg81366.html

You can vote yay or nay for each possible list syntax individually.

Voting will close 2017-02-10 16:00 (UTC).

Thanks to @WesNetmo for pushing me to bring this to vote. The
community (or at least a sample from twitter) seems to want this
(https://twitter.com/sammyk/status/824298464456691712) so here we go!
:)

Thanks,
Sammy Kaye Powers
sammyk.me

Hello my internals friends!

After a 2 week (ahem... 1 year & 3 month - sorry for dropping the
ball) discussion period, the "Trailing commas in all list syntax" RFC
is now open for voting.

See the discussion here:
internals@lists.php.net/msg81366.html" rel="nofollow" target="_blank">https://www.mail-archive.com/internals@lists.php.net/msg81366.html

You can vote yay or nay for each possible list syntax individually.

Voting will close 2017-02-10 16:00 (UTC).

Thanks to @WesNetmo for pushing me to bring this to vote. The
community (or at least a sample from twitter) seems to want this
(https://twitter.com/sammyk/status/824298464456691712) so here we go!
:)

I've voted based on the following rule:

• Vote yes if the list is enclosed in (), [] or {}.
• Vote no otherwise.

I can see how it is useful in some cases, but I really don't like the idea
of having a comma followed by a semicolon, as is the case for the class
member declarations...

Nikita

After a 2 week (ahem... 1 year & 3 month - sorry for dropping the
ball) discussion period, the "Trailing commas in all list syntax" RFC
is now open for voting.

See the discussion here:
internals@lists.php.net/msg81366.html" rel="nofollow" target="_blank">https://www.mail-archive.com/internals@lists.php.net/msg81366.html

You can vote yay or nay for each possible list syntax individually.

Voting will close 2017-02-10 16:00 (UTC).

Thanks to @WesNetmo for pushing me to bring this to vote. The
community (or at least a sample from twitter) seems to want this
(https://twitter.com/sammyk/status/824298464456691712) so here we go!
:)

Generally, I don't like these trailing commas, but indeed there is some
value for lists that often have to be split across multiple lines. As
such, my threshold is whether splitting certain lists across multiple
lines is sensible. In my opinion, the following lists should usually be
one-liners (because having too many items in the list would be a general
design issue):

• function/method parameters
• interface implementations on a class
• trait implementations on a class
• class member list (I prefer a distinct statement for each member)
• inheriting variables from the parent scope in anonymous functions

An improvement might be allowing trailing commas in grouped namespaces,
but I haven't enough experience with these to form an opinion.

The only other case where allowing trailing commas would make sense, in
my opinion, are function/methods arguments – there is, unfortunately
for me, no explicit voting option.

--
Christoph M. Becker

After a 2 week (ahem... 1 year & 3 month - sorry for dropping the
ball) discussion period, the "Trailing commas in all list syntax" RFC
is now open for voting.

See the discussion here:
internals@lists.php.net/msg81366.html" rel="nofollow" target="_blank">https://www.mail-archive.com/internals@lists.php.net/msg81366.html

You can vote yay or nay for each possible list syntax individually.

Voting will close 2017-02-10 16:00 (UTC).

Thanks to @WesNetmo for pushing me to bring this to vote. The
community (or at least a sample from twitter) seems to want this
(https://twitter.com/sammyk/status/824298464456691712) so here we go!
:)

I would have voted yes, but I missed the vote.
Unless there is critical reason not to implement it that I don't know of, I
prefer to allow
trailing commas like array universally/consistently.

Regards,

--
Yasuo Ohgaki
yohgaki@ohgaki.net