Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:59690
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.212.170 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <4F84C76A.9090104@sugarcrm.com>
References: <CAF+90c_hj7PrWbm42dj0DhN7Ygmc=vckP=bcYfsVuod4=RrOQQ@mail.gmail.com>
	<4F84C76A.9090104@sugarcrm.com>
Date: Tue, 10 Apr 2012 17:08:42 -0700
Message-ID: <CAKOpQSwsU-wq5fn=fKB1NNXKqVKCXGTe-3v74n1Kz2cSunaN7Q@mail.gmail.com>
To: Stas Malyshev <smalyshev@sugarcrm.com>
Cc: Nikita Popov <nikita.ppv@googlemail.com>, PHP internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary=f46d04389347c3d30c04bd5c0a00
Subject: Re: [PHP-DEV] Allow non-variable arguments to empty()
From: kris.craig@gmail.com (Kris Craig)

--f46d04389347c3d30c04bd5c0a00
Content-Type: text/plain; charset=ISO-8859-1

On Tue, Apr 10, 2012 at 4:51 PM, Stas Malyshev <smalyshev@sugarcrm.com>wrote:

> Hi!
>
> > Currently the empty() language construct only works on variables. You
> > can write if (empty($array)) but not empty if (empty(getSomeArray()).
> >
> > The original reason for this restriction probably is that - in a way -
> > it "doesn't make sense" to pass anything but a variable to empty() as
> > you could just use the ! operator in this case. if
> > (empty(getSomeArray())) is logically equivalent to if
> > (!getSomeArray()).
>
> Don't see any problem with that, looks good. Please add some tests
> (including function calls and some expressions) and make a pull request.
> --
> Stanislav Malyshev, Software Architect
> SugarCRM: http://www.sugarcrm.com/
> (408)454-6900 ext. 227
>
> --
> PHP Internals - PHP Runtime Development Mailing List
> To unsubscribe, visit: http://www.php.net/unsub.php
>
>
Err isn't this something that should go through the RFC process first?  I
think it's a good idea and I'll probably vote for it, but as I understand
the RFC process was created specifically for stuff like this.

--Kris

--f46d04389347c3d30c04bd5c0a00--