Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:43093
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 74.125.46.28 as permitted sender)
DomainKey-Status: bad
DomainKey-Signature: a=rsa-sha1; c=nofws;
        d=gmail.com; s=gamma;
        h=mime-version:in-reply-to:references:date:message-id:subject:from:to
         :cc:content-type;
        b=gLNK9gzn+Q7e2GGTo84deKpOeZQQBOxHIdWDEdaG1CepcA0bPPcByBvwyydJcz3wd8
         L2BlXZc31e1sQVVcHj47NojVNQRT2VKBAicKjC6YNNjaV5ZJSHr2yybL12oMCrDQ/aBo
         CQqBwKe9JRjXXt1qJhzZQ/VQlIKbJAaWRLGbI=
MIME-Version: 1.0
In-Reply-To: <1234963004.5869.6.camel@goldfinger>
References: <7dd2dc0b0902172008g5a634577pad280d1f0415453b@mail.gmail.com>
	 <499BC82B.2000000@daylessday.org> <1234963004.5869.6.camel@goldfinger>
Date: Wed, 18 Feb 2009 07:31:56 -0700
Message-ID: <7dd2dc0b0902180631q359a189aj5e4c54ef96d7f8e0@mail.gmail.com>
To: =?UTF-8?Q?Johannes_Schl=C3=BCter?= <johannes@php.net>
Cc: Antony Dovgal <tony@daylessday.org>, internals@lists.php.net
Content-Type: multipart/alternative; boundary=00221532ce5c27cfa404633248b2
Subject: Re: [PHP-DEV] zend_call_method() - support for up to 4 parameters
From: quickshiftin@gmail.com (Nathan Nobbe)

--00221532ce5c27cfa404633248b2
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

On Wed, Feb 18, 2009 at 6:16 AM, Johannes Schl=C3=BCter <johannes@php.net> =
wrote:

> But I don't think that a new limitation is any better: Tomorrow we have
> to change it again as somebody has a reason to use 5 parameters, so if
> it is changed it should  be changed to take any number of arguments and
> no fixed limit.... (or add a new API for that)


that does seem ideal to me, but in the last thread, marcus was clear that a=
n
emalloc / va_list solution was unacceptable.  that was last january.. has
anything changed in that regard since?

also, it sounded like he was saying a simple solution for up to 4 parameter=
s
would be alright. mainly i thought it would be nice to get it into the code
for 5.3 before we'd have to wait for another major release in order to get
something included.

yes, perhaps the function would need updating again in the future, but sinc=
e
people are using macros rather than zend_call_method directly it wouldnt be
too bad right?

-nathan

--00221532ce5c27cfa404633248b2--