Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:72210
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 74.125.82.170 as permitted sender)
Message-ID: <52F1543F.4010805@gmail.com>
Date: Tue, 04 Feb 2014 20:57:35 +0000
User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:24.0) Gecko/20100101 Thunderbird/24.2.0
MIME-Version: 1.0
To: internals@lists.php.net
References: <52EE2B1D.7000307@pthreads.org>	<52EEA7BE.4050500@sugarcrm.com>	<52EEF636.7010505@pthreads.org>	<52EF5588.6010902@sugarcrm.com>	<CA+9eiLvsF5Uu--TaZ9vDOn4iifK19Jdqi+iSrsxgZK8_Hx9Uug@mail.gmail.com>	<52F01109.3020705@sugarcrm.com> <CA+9eiLvrbWnmWjh3ovQbEpO9p5v=NwJf9GnDUie_wPE9MbpdoA@mail.gmail.com>
In-Reply-To: <CA+9eiLvrbWnmWjh3ovQbEpO9p5v=NwJf9GnDUie_wPE9MbpdoA@mail.gmail.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Subject: Re: [PHP-DEV] RFC: expectations/assertions
From: rowan.collins@gmail.com (Rowan Collins)

On 04/02/2014 05:36, Dmitry Stogov wrote:
>> >BTW, current patch has zend.assertions as INI_SYSTEM, which means the
>> >whole server can either have assertions on or off.
> It's INI_SYSTEM on purpose. zend.assertions is a 3 way switch that may
> disable code generation for assert().
> Changing it after PHP files are already cached won't make any effect.
>
>

Incidentally, if this is true, the introduction in the RFC is incorrect, 
as it currently states:

 > assertions can therefore be disabled and enabled via the PHP_INI_ALL 
configuration setting zend.assertions

The explanation of *why* it would be INI_SYSTEM could also be added there.

-- 
Rowan Collins
[IMSoP]