Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:56194
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain googlemail.com designates 209.85.161.42 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <op.v4ogjdwiexzowv@pc-kubelik.iinfo.local>
References: <op.v4ogjdwiexzowv@pc-kubelik.iinfo.local>
Date: Wed, 9 Nov 2011 16:24:33 +0100
Message-ID: <CAF+90c89KJow6yP4y1fsjvkkZxERhnEpGEyLWRQswD8fwDx0nQ@mail.gmail.com>
To: =?ISO-8859-1?Q?Miroslav_Kubel=EDk?= <koubel@php.net>
Cc: internals@lists.php.net
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: quoted-printable
Subject: Re: [PHP-DEV] Re: Ternary operator performance improvements
From: nikita.ppv@googlemail.com (Nikita Popov)

Hi Miroslav!

Yes, the ternary patch fixes that problem too :)

Nikita

2011/11/9 Miroslav Kubel=EDk <koubel@php.net>:
> On 10/14/2011 10:08 PM, Arnaud Le Blanc wrote:
>>
>> Hi,
>>
>> I've already posted this patch and it has since been reviewed and
>> improved.
>> I'm re-posting it for discussion before eventually commiting it.
>>
>> The ternary operator always copies its second or third operand, which is
>> very
>> slow compared to an if/else when the operand is an array for example: ..=
.
>
> Same "copy always" behavior is made with string in ternary operator, and
> it can be
> also very annoying e.g:
>
> $content =3D
> file_get_contents('http://www.w3.org/TR/2011/WD-html-markup-20110525/spec=
.html');
> $test =3D $content !=3D=3D false ? $content : '';
>
> Is this case also solved with your patch or huge strings are still proble=
m
> with a ternary "copy always" functionality? Maybe it's more rare case the=
n
> array.
>
> P.S: Sorry I don't test it directly with beta2, I have no place to test.
>
> M. Kubelik
>
>
> --
> PHP Internals - PHP Runtime Development Mailing List
> To unsubscribe, visit: http://www.php.net/unsub.php
>
>