Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:56188
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: unknown (pb1.pair.com: domain php.net does not designate 213.151.94.173 as permitted sender)
Content-Type: text/plain; charset=utf-8; format=flowed; delsp=yes
To: internals@lists.php.net
Date: Wed, 09 Nov 2011 12:51:03 +0100
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Message-ID: <op.v4ogjdwiexzowv@pc-kubelik.iinfo.local>
User-Agent: Opera Mail/11.52 (Win32)
Subject: Re: Ternary operator performance improvements 
From: koubel@php.net (=?utf-8?Q?Miroslav_Kubel=C3=ADk?=)

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 =
file_get_contents('http://www.w3.org/TR/2011/WD-html-markup-20110525/spec.html');
$test = $content !== false ? $content : '';

Is this case also solved with your patch or huge strings are still problem
with a ternary "copy always" functionality? Maybe it's more rare case then
array.

P.S: Sorry I don't test it directly with beta2, I have no place to test.

M. Kubelik