Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:72240
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: error (pb1.pair.com: domain zend.com from 74.125.82.49 cause and error)
MIME-Version: 1.0
In-Reply-To: <CAGa2bXa+TOORvK7Jf80Dm6wRNQDnVEEyj3yovdL2zbwYPPZV8A@mail.gmail.com>
References: <CAGa2bXYHP5D70bDBhbWzE7-B0s2kgk7DmAZdFhXkLqMJQqKV5g@mail.gmail.com>
	<52EF4BF8.60005@sugarcrm.com>
	<CAGa2bXacCxnmcAPZjh-YgTNocMjEpu_Uoc4npfNcKVM2WA2Tow@mail.gmail.com>
	<CAGa2bXZt7n8aqA=VrBKsqBzFiusGZP7aECPO-7xQher4+uVLWg@mail.gmail.com>
	<CA+9eiLsC1nXFBBQ4bDAHNhMFJPg04UTgxG2jjAxdX4jTG7O0xA@mail.gmail.com>
	<CAGa2bXa+TOORvK7Jf80Dm6wRNQDnVEEyj3yovdL2zbwYPPZV8A@mail.gmail.com>
Date: Wed, 5 Feb 2014 10:38:34 +0400
Message-ID: <CA+9eiLs3p3jqy6TMy10GmJfEe4k7DoXUJUvAJ01MA1y-svDs+g@mail.gmail.com>
To: Yasuo Ohgaki <yohgaki@ohgaki.net>
Cc: "internals@lists.php.net" <internals@lists.php.net>
Content-Type: multipart/alternative; boundary=f46d044283b085a14f04f1a30175
Subject: Re: [PHP-DEV] Declare minimum PHP version required?
From: dmitry@zend.com (Dmitry Stogov)

--f46d044283b085a14f04f1a30175
Content-Type: text/plain; charset=UTF-8

On Wed, Feb 5, 2014 at 1:19 AM, Yasuo Ohgaki <yohgaki@ohgaki.net> wrote:

> Hi Dmitry,
>
> On Wed, Feb 5, 2014 at 3:30 AM, Dmitry Stogov <dmitry@zend.com> wrote:
>
>> I don't think PHP needs new language constructs for assertions.
>> Actually, the code from your example may be implemented as two calls to
>> assert().
>>
>> assert(extension_loaded('foo'), 'You need foo module');
>> assert(extension_loaded('bar'), 'You need bar module');
>>
>> I think it's much clear then inventing new syntax.
>> With Joe's proposal these asserts might be completely eliminated.
>> And it's backward and forward compatible :)
>>
>> BTW: I didn't follow all the discussion, so I may miss some points.
>>
>
> Thank you for reply.
>
> Since new assert() could be eliminate runtime overheads, it may be the
> way to go. Regarding assert(), it seems current assert() is inconsistent
> for function calls.
>
> Function call works
> php > function f() {return FALSE;}
> php > assert(f());
>
> Warning: assert(): Assertion failed in php shell code on line 1
>
> Closure does not
> php > assert(function() {return FALSE;});
> php >
>
> Closure is a valid expression and it should be allowed. IMO.
>

Yeah, closure is a valid expression but it doesn't tell that closure has to
be called.

assert("f"); // doesn't call f() as well.

So I don't see inconsistency.


> Allowing callable as 1st parameter makes behavior consistent.
> Closure is useful to keep clean name space.
> Could you consider this for new assert?
>

I'm not sure it's related to assert(). PHP misses construct to create and
call closure in one expression.
So currently we can't write:

assert(function() {return FALSE;}());

but we can write:

$f = function() {return FALSE;};
assert($f());

Thanks. Dmitry.


> Regards,
>
> --
> Yasuo Ohgaki
> yohgaki@ohgaki.net
>

--f46d044283b085a14f04f1a30175--