Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:58962
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 74.125.82.170 as permitted sender)
MIME-Version: 1.0
Sender: patrick.allaert@gmail.com
In-Reply-To: <CAF+90c-L0yKM61Om4tUFdgzfrRu1m7S5wZgh1ZZBiiOgWH+Efw@mail.gmail.com>
References: <CAFst3Wptf7pmHwdWPwifbMpfCrUhHog5QqQtkyvunfm+S6R6XA@mail.gmail.com>
	<CAF+90c-L0yKM61Om4tUFdgzfrRu1m7S5wZgh1ZZBiiOgWH+Efw@mail.gmail.com>
Date: Thu, 15 Mar 2012 17:22:28 +0100
Message-ID: <CACwt+Jd2Z7Wyh78Xn_=NMVAn_H57Pw48ARgFo6o9yw-xDVzYAQ@mail.gmail.com>
To: Nikita Popov <nikita.ppv@googlemail.com>
Cc: Klaus Silveira <klaussilveira@php.net>, PHP Internals <internals@lists.php.net>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable
Subject: Re: [PHP-DEV] Small question about performance
From: patrickallaert@php.net (Patrick ALLAERT)

2012/3/15 Nikita Popov <nikita.ppv@googlemail.com>:
> If I am understanding the text correctly it is saying that
> =C2=A0 =C2=A0$f1 =3D f1();
> =C2=A0 =C2=A0$f2 =3D f2($f1);
> =C2=A0 =C2=A0$f3 =3D f3($f2);
> is using more memory than
> =C2=A0 =C2=A0$f3 =3D f3(f2(f1()));
>
> For me this doesn't make any sense. In the latter case PHP will also
> create temporary variables to store the return values. There should be
> no difference in memory consumption.

It does make sense to me.

In the first case, when calling f3(), $f1 is still referenced.
In the second case, when calling f3(), the result of f2() is
referenced, but there is no more active reference to the result of
f1().

Regarding the original problem:
foreach($a as $key =3D> $val) {
    $a[$key] =3D someLong(functionCalls(hereThat($spanOver85Chars)));
}

Sounds easier to split over lines without temporary zvals:
foreach($a as $key =3D> $val) {
    $a[$key] =3D someLong(
        functionCalls(
            hereThat(
                $spanOver85Chars
            )
        )
    );
}