Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:109843
MIME-Version: 1.0
References: <CAPyj-LDo0AafB9FZoxUHU35Y7PAU+DYb4qGm+KV2-eF5KvNmng@mail.gmail.com>
In-Reply-To: <CAPyj-LDo0AafB9FZoxUHU35Y7PAU+DYb4qGm+KV2-eF5KvNmng@mail.gmail.com>
Date: Sat, 25 Apr 2020 12:03:31 +0100
Message-ID: <CA+kxMuRo-oVGYCuFRsB8mag=x6W57Yy0CmY6rXeRQ55pKqLsew@mail.gmail.com>
To: Ilija Tovilo <tovilo.ilija@gmail.com>
Cc: PHP internals <internals@lists.php.net>
Content-Type: text/plain; charset="UTF-8"
Subject: Re: [PHP-DEV] [VOTE] match expression
From: Danack@basereality.com (Dan Ackroyd)

On Sat, 25 Apr 2020 at 11:40, Ilija Tovilo <tovilo.ilija@gmail.com> wrote:
>
> Hi internals
>
> I have opened the voting on the match expression RFC. It will end on
> 9th of May, 2020.
> https://wiki.php.net/rfc/match_expression
>
> Here are the last changes:


Hi Ilija,

I like this idea, and would like to see 'match' in PHP. At the same
time, is there any need to have the vote right now? The deadline for
PHP 8 feature freeze is July 27 2020.

There were changes to the proposal overnight, which people have not
even had a chance to read, let alone think about. To me it feels like
the RFC is being rushed to a vote, with respect to the recent changes
and the technical problem you describe.

> Unfortunately, we discovered some technical difficulties (memory
> leaks) which are very hard to solve. Nikita is experimenting on a
> possible solution.

It also seems the voting options were also changed last night.

- The vote is a straight Yes/No vote for accepting the RFC and merging
the patch.

+ "Would you like to add match expressions to the language?"
+ "Should the semicolon for match in statement form be optional?"
+ "Should we allow dropping (true) condition?"

I think it would be better to continue the conversation and work
through the trade-offs when voting isn't active, rather than just
hoping people pick the correct choices, or have a contentious
time-limited discussion.

cheers
Dan
Ack