On Wed, Jul 23, 2014 at 10:45 PM, Bob Weinand <bobw...@hotmail.com> wrote:
> Am 23.7.2014 um 22:33 schrieb Stas Malyshev <smalys...@sugarcrm.com>: > > Hi! > > > >> It was a restriction to not support arrays in constant context. It seems > >> like nobody can remember why it was introduced. > > > > My vague recollection is that it had some troubles with keeping > > refcounts consistent, esp. withing bytecode caching context, but it may > > be a false memory :) > > That doesn't really make much sense to me, at least not with current > engine... > As said, I've tested it and didn't find issues. Maybe I just test the > wrong things… > > >> However, I think it's too dangerous to break it in last minute before > >> release. > > > > We definitely need to fix the WTF with "no runtime use" for defined > > constant and the segfault before the release. I think for the arrays, if > > we can't have it working properly we'd better not have array support > > there for 5.6.0 and fix it in 5.6.1 than have this weird "no runtime > > use" thing. > > Well, we still could fix it now, AFAIK, we will still have another RC now > and when we fix it only for 5.6.1, there also will only be one RC in > between. > That's no gain in testing time etc.. > > Bob > > > -- > > Stanislav Malyshev, Software Architect > > SugarCRM: http://www.sugarcrm.com/ > > -- > PHP Internals - PHP Runtime Development Mailing List > To unsubscribe, visit: http://www.php.net/unsub.php > > yeah, there will be another RC for sure, but we have to have some resulotion for that. I think it would be safer to remove stuff (even if just the array support) than trying to fix in in a hurry, as it would probably just cause shipping the final with some other bug(s) or having a couple of other RCs. -- Ferenc Kovács @Tyr43l - http://tyrael.hu
