Newsgroups: php.internals Path: news.php.net Xref: news.php.net php.internals:52796 Return-Path: Mailing-List: contact internals-help@lists.php.net; run by ezmlm Delivered-To: mailing list internals@lists.php.net Received: (qmail 85231 invoked from network); 2 Jun 2011 17:10:54 -0000 Received: from unknown (HELO lists.php.net) (127.0.0.1) by localhost with SMTP; 2 Jun 2011 17:10:54 -0000 Authentication-Results: pb1.pair.com header.from=ilia@prohost.org; sender-id=pass Authentication-Results: pb1.pair.com smtp.mail=ilia@prohost.org; spf=pass; sender-id=pass Received-SPF: pass (pb1.pair.com: domain prohost.org designates 209.85.214.42 as permitted sender) X-PHP-List-Original-Sender: ilia@prohost.org X-Host-Fingerprint: 209.85.214.42 mail-bw0-f42.google.com Received: from [209.85.214.42] ([209.85.214.42:34385] helo=mail-bw0-f42.google.com) by pb1.pair.com (ecelerity 2.1.1.9-wez r(12769M)) with ESMTP id 26/31-11525-D14C7ED4 for ; Thu, 02 Jun 2011 13:10:53 -0400 Received: by bwz18 with SMTP id 18so1257206bwz.29 for ; Thu, 02 Jun 2011 10:10:50 -0700 (PDT) MIME-Version: 1.0 Received: by 10.204.84.27 with SMTP id h27mr965919bkl.158.1307034650754; Thu, 02 Jun 2011 10:10:50 -0700 (PDT) Received: by 10.204.72.3 with HTTP; Thu, 2 Jun 2011 10:10:50 -0700 (PDT) In-Reply-To: References: Date: Thu, 2 Jun 2011 19:10:50 +0200 Message-ID: To: Pierre Joye Cc: PHP Developers Mailing List Content-Type: text/plain; charset=ISO-8859-1 Subject: Re: [PHP-DEV] RFC: Zend Signal Handling From: ilia@prohost.org (Ilia Alshanetsky) Killing TSRMLS_FETCH is a noble goal, but let's keep it to once patch at a time please ;-) And for the record I am all for killing TSRMLS_FETCH. On Thu, Jun 2, 2011 at 6:04 PM, Pierre Joye 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 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 >