Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:43090
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: unknown (pb1.pair.com: domain php.net does not designate 83.243.58.134 as permitted sender)
To: Antony Dovgal <tony@daylessday.org>
Cc: Nathan Nobbe <quickshiftin@gmail.com>, internals@lists.php.net
In-Reply-To: <499BC82B.2000000@daylessday.org>
References: <7dd2dc0b0902172008g5a634577pad280d1f0415453b@mail.gmail.com>
	 <499BC82B.2000000@daylessday.org>
Content-Type: text/plain
Date: Wed, 18 Feb 2009 14:16:44 +0100
Message-ID: <1234963004.5869.6.camel@goldfinger>
Mime-Version: 1.0
Content-Transfer-Encoding: 7bit
Subject: Re: [PHP-DEV] zend_call_method() - support for up to 4 parameters
From: johannes@php.net (Johannes =?ISO-8859-1?Q?Schl=FCter?=)

On Wed, 2009-02-18 at 11:34 +0300, Antony Dovgal wrote:
> > recently, working on an extension, i wanted to call a method w/ 3 params,
> > and as you know, zend_call_method only supports 2 parameters at most.  i
> > came across this thread in the archives,
[...]
> What happened to call_user_function() ?
> Why break the API and make extension maintainers use even more #if's ?


The benefit of zend_call_method is that you can keep the function
pointer and so the lookup has not to do the lookup every time it is
called which is quite nice.

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)

johannes