Re: TSRMLS_FETCH crusade (was: Re: [PHP-CVS] com php-src: Eliminate another TSRMLS_FETCH())

From: Date: Wed, 18 Dec 2013 13:06:44 +0000
Subject: Re: TSRMLS_FETCH crusade (was: Re: [PHP-CVS] com php-src: Eliminate another TSRMLS_FETCH())
References: 1 2  Groups: php.cvs php.internals 
Request: Send a blank email to php-cvs+get-74499@lists.php.net to get a copy of this message 
On Wed, 18 Dec 2013, Mike wrote:

> On Wed, 2013-12-18 at 06:25 +0000, Kalle Sommer Nielsen wrote:
> > Commit:    d14670ccdb2782abaaaa721d1c379138fa64aae6
> > Author:    Kalle Sommer Nielsen <kalle@php.net>         Wed, 18 Dec 2013 07:25:05
> > +0100
> > Parents:   1433dec2d0c00fcda5bebaeebefca8264deb912a
> > Branches:  master
> > 
> > Link:       http://git.php.net/?p=php-src.git;a=commitdiff;h=d14670ccdb2782abaaaa721d1c379138fa64aae6
> > 
> > Log:
> > Eliminate another TSRMLS_FETCH() in i_zend_is_true()
> > 
> > # Affected extensions have all been updated, ext/opcache and ext/zip
> > # both have macros for cross version compatibility
> 
> What's with all the other extensions? PECL or wild? That TSRMLS_FETCH
> crusade seems pretty worthless as long as it is found in _emalloc etc.

A much nicer thing would be to just get rid of TS and TSRMLS in the 
first place across the board.

cheers,
Derick

Thread (5 messages)

« previous php.cvs (#74499) next »