Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:86813
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.217.179 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <CABBJUpeaVXXJ-JJDP+oHUamuDGwM87Myvm4_0qXAh2cZn8rg3A@mail.gmail.com>
References: <CAGa2bXa9x_BK9O78N_yvAVj7M5V0UEGsy3ch-VOimaqGaKUA3w@mail.gmail.com>
	<CABBJUpeaVXXJ-JJDP+oHUamuDGwM87Myvm4_0qXAh2cZn8rg3A@mail.gmail.com>
Date: Tue, 23 Jun 2015 11:21:34 -0400
Message-ID: <CAAyV7nHo2jWSNYL-BQc-yg2SoTzOpNFvcmLNpH9vhL9w3_ab7w@mail.gmail.com>
To: Xinchen Hui <laruence@php.net>
Cc: Yasuo Ohgaki <yohgaki@ohgaki.net>, "internals@lists.php.net" <internals@lists.php.net>
Content-Type: text/plain; charset=UTF-8
Subject: Re: [PHP-DEV] Optimizing php_html_entities()
From: ircmaxell@gmail.com (Anthony Ferrara)

On Tue, Jun 23, 2015 at 10:33 AM, Xinchen Hui <laruence@php.net> wrote:
> Hey:
>
> On Tue, Jun 23, 2015 at 7:37 PM, Yasuo Ohgaki <yohgaki@ohgaki.net> wrote:
>> Hi all,
>>
>> I'm trying to optimize php_html_entities().
>> Since htmlspecialchars()/htmlentities() are sensitive function, I would
>> like to ask comments before merge.
>>
>> https://github.com/php/php-src/pull/1356
>>
>> Current php_html_entities() convert int/float/etc to string, then convert
>> it.
>> int/float/etc is not required to be escaped. Optimize it by simply
>> converting them to string.
>>
>> Simple benchmark shows about 60% execution time reduction.
>>
>> [yohgaki@dev github-php-src]$ ./php-bin b.php
>> Time: 10.484607934952
>> [yohgaki@dev github-php-src]$ ./php-bin b.php
>> Time: 10.867615222931
>> [yohgaki@dev github-php-src]$ ./php.new b.php
>> Time: 3.9379420280457
>> [yohgaki@dev github-php-src]$ ./php.new b.php
>> Time: 4.0694711208344
>> [yohgaki@dev github-php-src]$ cat b.php
>> <?php
>> const LOOP=100000000;
>>
>> $start = microtime(true);
>> for ($i = 0; $i < LOOP; $i++) {
>> }
>> $loop_time = microtime(true) - $start;
>>
>> $start = microtime(true);
>> for ($i = 0; $i < LOOP; $i++) {
>> htmlspecialchars(123456790);
>> }
>> echo 'Time: '.(microtime(true) - $start - $loop_time)."\n";
> But passing an non-string to htmlspecialchars are not common used cases..
>
> "optimize" not common used cases... will bring nothing to us..

In addition, this breaks the contract, specifically when using scalar
types. Because you're no longer going to error when the contract is
broken (considering htmlspecialchars is documented as string:string).

Anthony