That would be a solution. The only thing is that this would be a break in BC for extensions using zend_alter_ini_entry.
I will test your patch with my webserver together with the @-Operator. ----- Uwe Schindler [EMAIL PROTECTED] - http://www.php.net NSAPI SAPI developer Bremen, Germany > -----Original Message----- > From: Jani Taskinen [mailto:[EMAIL PROTECTED] > Sent: Wednesday, September 05, 2007 2:06 PM > To: Stanislav Malyshev > Cc: internals@lists.php.net > Subject: [PHP-DEV] Re: [ZEND-ENGINE-CVS] cvs: ZendEngine2 / zend_ini.c > > Sorry for the top posting Derick. :) > > Stas, this patch solves the problems with @ and doing > "php_admin_value error_reporting <some value>": > > http://pecl.php.net/~jani/patches/zend_alter_ini_entry.patch > > I renamed zend_alter_ini_entry() to zend_alter_ini_entry_ex() and added > one more parameter to force change regardless of the modifiable value. > > --Jani > > > On Mon, 2007-09-03 at 15:20 +0300, Jani Taskinen wrote: > > See the 2nd last comment (by [EMAIL PROTECTED]): > > http://bugs.php.net/bug.php?id=41561 > > > > The handling of the @ operator is the issue here. Shouldn't that be > > turned on/off without touching the INI stuff anyway? (I mailed about > > this earlier too, nobody bothered to reply..) > > > > --Jani > > > > > > On Fri, 2007-08-31 at 10:27 -0700, Stanislav Malyshev wrote: > > > Could you explain what was the problem and how it was solved? > > > > > > Jani Taskinen wrote: > > > > jani Fri Aug 31 07:52:09 2007 UTC > > > > > > > > Modified files: > > > > /ZendEngine2 zend_ini.c > > > > Log: > > > > - Revert the revert: this is not causing any problems (or we have > lot bigger issues), the bug is elsewhere > > > > -- > Make me happy: http://pecl.php.net/wishlist.php/jani > > -- > PHP Internals - PHP Runtime Development Mailing List > To unsubscribe, visit: http://www.php.net/unsub.php -- PHP Internals - PHP Runtime Development Mailing List To unsubscribe, visit: http://www.php.net/unsub.php
