On Thu, Jun 2, 2011 at 7:10 PM, Ilia Alshanetsky <i...@prohost.org> wrote: > Killing TSRMLS_FETCH is a noble goal, but let's keep it to once patch > at a time please ;-)
I mean in this patch only. This patch adds a couple, so it can be done at the same time (afair these functions are not used heavily outside the engine). > And for the record I am all for killing > TSRMLS_FETCH. Same here (tls and some other ideas may help :) > On Thu, Jun 2, 2011 at 6:04 PM, Pierre Joye <pierre....@gmail.com> wrote: >> hi Ilia, >> >> I would suggest to kill the TSRMLS_FETCH while being at it. They are >> horribly slow and a couple of them can be replaced by the >> TSRMLS_DC/CC, if I'm not mistaken. >> >> For the windows side, I do not have the time to do the equivalent, so >> if you commit the patch to trunk first so I can fix the build >> accordingly and then merge, that would be easier for me :). >> >> Cheers, >> >> On Wed, Jun 1, 2011 at 12:30 AM, Ilia Alshanetsky <i...@prohost.org> wrote: >>> Since we are on the topic of reviewing past RFCs for 5.4, can we take >>> another look at the Zend Signals RFC: >>> >>> https://wiki.php.net/rfc/zendsignals >>> >>> The patch is solid (have been using it in production for quite some >>> time) and improvement is quite helpful, especially when APC is being >>> used. Are there any reasons not to apply this to 5.4? >>> >>> -- >>> PHP Internals - PHP Runtime Development Mailing List >>> To unsubscribe, visit: http://www.php.net/unsub.php >>> >>> >> >> >> >> -- >> Pierre >> >> @pierrejoye | http://blog.thepimp.net | http://www.libgd.org >> > -- Pierre @pierrejoye | http://blog.thepimp.net | http://www.libgd.org -- PHP Internals - PHP Runtime Development Mailing List To unsubscribe, visit: http://www.php.net/unsub.php
