Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:42999
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: error (pb1.pair.com: domain pooteeweet.org from 88.198.8.16 cause and error)
Cc: Daniel Convissor <danielc@analysisandsolutions.com>,
 PHP Internals List <internals@lists.php.net>
Message-ID: <51EC1113-B2D0-4F81-8055-C6D883B50934@pooteeweet.org>
To: Ilia Alshanetsky <ilia@prohost.org>
In-Reply-To: <9E2FED17-9EB8-421A-990D-D2EC789EAB2D@prohost.org>
Content-Type: text/plain; charset=US-ASCII; format=flowed; delsp=yes
Content-Transfer-Encoding: 7bit
Mime-Version: 1.0 (Apple Message framework v930.3)
Date: Wed, 11 Feb 2009 17:24:19 +0100
References: <ce6b011c0902091827q7ea25724u5f6c2de24c9c7713@mail.gmail.com> <20090211161147.GA13632@panix.com> <9E2FED17-9EB8-421A-990D-D2EC789EAB2D@prohost.org>
Subject: Re: [PHP-DEV] RFC for new INI's
From: mls@pooteeweet.org (Lukas Kahwe Smith)


On 11.02.2009, at 17:19, Ilia Alshanetsky wrote:

>
> On 11-Feb-09, at 11:11 AM, Daniel Convissor wrote:
>
>> Hi Eric:
>>
>> On Mon, Feb 09, 2009 at 09:27:03PM -0500, Eric Stewart wrote:
>>> http://wiki.php.net/rfc/newinis
>>
>> Thanks for the work.
>>
>> I noticed the actual directives in the files are the same.  I guess  
>> you
>> just focused on the Quick Reference section and getting the  
>> comments set
>> up the way you want.
>>
>> Line 488 in prod and 483 in dev need to have the link commented out.
>>
>> error_reporting:  should be E_ALL in production.  Well written code
>> should not be generating notices, so notices indicate a problem that
>> needs fixing.
>
> I think in dev environment it makes sense to set error reporting  
> level to E_ALL | E_STRICT


Right. Currently E_DEPRECATED is also part of E_ALL. Maybe it should  
not. In dev env it should of course be enabled.

regards,
Lukas Kahwe Smith
mls@pooteeweet.org