On Thu, 2008-04-17 at 17:59 +0100, Scott MacVicar wrote: > Hi Richard, > > The patch would fix it in this case but there is an underlying change > that needs sorted. > > zend_ini_string()'s behaviour was changed to fix > http://bugs.php.net/bug.php?id=42657 so that if the default value was > NULL it would then return an empty string instead. > > http://cvs.php.net/viewvc.cgi/ZendEngine2/zend_ini.c?r1=1.39.2.2.2.18.2.6&r2=1.39.2.2.2.18.2.7 > > This broke the places that were relying on INI_STR returning the NULL?
Propably, but INI_STR is evil and should never be used anyway. > Should this fix be backed out and change ini_get and ini_set to deal > with this for the userland? Hell no. Fix places using INI_STR incorrectly instead. (as in: make them NOT use INI_STR.. :) --Jani -- PHP Internals - PHP Runtime Development Mailing List To unsubscribe, visit: http://www.php.net/unsub.php
