Re: com php-src: Use new param API in standard: ext/standard/browscap.c ext/standard/cyr_convert.c ext/standard/datetime.c ext/standard/dl.c
ext/standard/dns.c ext/standard/fsock.c ext/standard/metaphone.c ext/standard/type.c ext/standard/uniqid
| From: | Andrea Faulds | Date: | Tue, 03 Jan 2017 03:53:03 +0000 |
| Subject: | Re: com php-src: Use new param API in standard: ext/standard/browscap.c ext/standard/cyr_convert.c ext/standard/datetime.c ext/standard/dl.c ext/standard/dns.c ext/standard/fsock.c ext/standard/metaphone.c ext/standard/type.c ext/standard/uniqid |
||
| References: | 1 2 | Groups: | php.internals |
| Request: | Send a blank email to internals+get-97503@lists.php.net to get a copy of this message | ||
Hi,
Derick Rethans wrote:
I thought the idea was to only for this for functions that would be called often as the win is so little?It is. Using the new parameter API increases the executable size significantly. When I removed the conditional compilation of the new API (because nobody was disabling it), I also removed the comment explaining it should be used only for hot code. That might have been a mistake. -- Andrea Faulds https://ajf.me/