Re: TSRMLS_FETCH crusade (was: Re: [PHP-CVS] com php-src: Eliminate another TSRMLS_FETCH())
| From: | Derick Rethans | 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 internals+get-70730@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