Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:39226
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: error (pb1.pair.com: domain realplain.com from 209.235.152.153 cause and error)
Message-ID: <014401c8ecca$4047b7d0$0201a8c0@pc1>
To: <internals@lists.php.net>, "Christian Schneider" <cschneid@cschneid.com>
References: <01b401c8ec13$28cd3510$0201a8c0@pc1> <18.5D.12534.2EF17884@pb1.pair.com>
Date: Wed, 23 Jul 2008 08:44:00 -0500
MIME-Version: 1.0
Content-Type: text/plain;
	charset="iso-8859-1"
Content-Transfer-Encoding: 7bit
Subject: Re: [PHP-DEV] Re: [PATCH] New functions: array_replace[_recursive]
From: php_lists@realplain.com ("Matt Wilmas")

Hi Christian,

----- Original Message -----
From: "Christian Schneider"
Sent: Wednesday, July 23, 2008

> Matt Wilmas wrote:
> > array_replace() is like the + operator applied to arrays, except that it
> > WILL overwrite ("replace") existing entries.
>
> Excuse my ignorance but what's the difference between
> $array = array_replace($array1, $array2);
> and
> $array = $array2 + $array1;
> apart from different order of the entries in the resulting array?

The order of the entries is the only difference, I believe. :-)  I think
that's desirable however, and having the non-recursive version makes it
match array_merge(), etc.  As for bloat or something, there's virtually no
extra code: only PHP_FUNCTION() and then in the wrapper function it just
uses zend_hash_merge() just like the + operator, except with its "overwrite"
param set to 1.

> > array_replace_recursive() will do the same except that it becomes
> > recursive only when both the destination and source entries are
> > arrays, otherwise the new source entry still replaces any existing
> > one.
>
> Hmm, generic enough to be included?

What Lukas said. :-)

> - Chris

- Matt