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: Date: Sun, 01 Jan 2017 17:43:58 +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 3  Groups: php.cvs php.internals 
Request: Send a blank email to internals+get-97499@lists.php.net to get a copy of this message
On Sun, Jan 1, 2017 at 4:51 AM, Christoph M. Becker <cmbecker69@gmx.de> wrote: > On 01.01.2017 at 13:36, 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? > > The performance improvement can be considerable[1], but the downside is > a larger executable. > Also leads to better readability, IMO. If the hit to executable size is too much, I can stop (kinda have stopped already, having finished off standard with the exception of a few edge cases). I just had a few days of unfocused energy on hand. -Sara

« previous php.internals (#97499) next »