Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:83379
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.220.170 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <DUB113-W13349F3B070957C64B0AF18F92B0@phx.gbl>
References: <DUB113-W13349F3B070957C64B0AF18F92B0@phx.gbl>
Date: Sat, 21 Feb 2015 06:20:35 +0000
Message-ID: <CAPwsocHXu+cXKs1ospdNRAAvRu1Qp3bb7q=mfvr6MxmsxDJWXw@mail.gmail.com>
To: Thomas Punt <tpunt@hotmail.co.uk>
Cc: "internals@lists.php.net" <internals@lists.php.net>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable
Subject: Re: [PHP-DEV] [RFC] Make empty() a Variadic
From: leight@gmail.com (Leigh)

Hey Tom,

Patch looks solid (basically the same as the isset logic with OR
instead of AND). I think it's fairly sane to have this feature because
it compliments isset functionality (although I dislike "empty"
personally - consistency is nice)

No RFC would be complete without a complaint about naming.
"empty_expressions" ... I'll give you an empty expression!

But overall +1 on functionality and patch.

Cheers!

Leigh.

On 21 February 2015 at 05:11, Thomas Punt <tpunt@hotmail.co.uk> wrote:
> Hello Internals!
> The following RFC aims to make empty() have a variable arity: https://wik=
i.php.net/rfc/variadic_empty. This is a simple feature that enables for a s=
hort-hand notation of checking multiple expressions for emptiness (which is=
 a pretty common thing to do).
> I have avoided including the is_*() functions into this RFC because my ma=
in priority is to get this particular feature into PHP 7.0 before feature f=
reeze. Provided I have some free time over the next week, I'll write up ano=
ther RFC + patch to cover the is_*() functions.
> Thanks,Tom