Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:72215
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.217.176 as permitted sender)
MIME-Version: 1.0
Sender: yohgaki@gmail.com
In-Reply-To: <CA+9eiLsC1nXFBBQ4bDAHNhMFJPg04UTgxG2jjAxdX4jTG7O0xA@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>
Date: Wed, 5 Feb 2014 06:19:08 +0900
Message-ID: <CAGa2bXa+TOORvK7Jf80Dm6wRNQDnVEEyj3yovdL2zbwYPPZV8A@mail.gmail.com>
To: Dmitry Stogov <dmitry@zend.com>
Cc: "internals@lists.php.net" <internals@lists.php.net>
Content-Type: multipart/alternative; boundary=047d7b3a894635bdd704f19b3342
Subject: Re: [PHP-DEV] Declare minimum PHP version required?
From: yohgaki@ohgaki.net (Yasuo Ohgaki)

--047d7b3a894635bdd704f19b3342
Content-Type: text/plain; charset=UTF-8

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.
Allowing callable as 1st parameter makes behavior consistent.
Closure is useful to keep clean name space.
Could you consider this for new assert?

Regards,

--
Yasuo Ohgaki
yohgaki@ohgaki.net

--047d7b3a894635bdd704f19b3342--