Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:55910
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.214.42 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <4EA3B952.1010608@sugarcrm.com>
References: <CAL5nwZ4G39cnsbOhayUSqROqg1+CgLszyNoaqf+5W1h6pu6QfQ@mail.gmail.com>
	<4EA3B952.1010608@sugarcrm.com>
Date: Sun, 23 Oct 2011 09:39:23 -0600
Message-ID: <CAL5nwZ5JeQUEYCvY4XxSdMkCyPDiQ+AHL9Q9F50pje2XiLz3CQ@mail.gmail.com>
To: Stas Malyshev <smalyshev@sugarcrm.com>
Cc: "internals@lists.php.net" <internals@lists.php.net>
Content-Type: multipart/alternative; boundary=0015175707e44c203704aff91cc5
Subject: Re: [PHP-DEV] Revisit: zend_call_method() - n number of arguments
From: quickshiftin@gmail.com (Nathan Nobbe)

--0015175707e44c203704aff91cc5
Content-Type: text/plain; charset=UTF-8

On Sun, Oct 23, 2011 at 12:50 AM, Stas Malyshev <smalyshev@sugarcrm.com>wrote:

> Hi!
>
>
> On 10/22/11 2:36 AM, Nathan Nobbe wrote:
>
>> Hi,
>>
>> Another old issue I'd like to rehash with the upcoming release around the
>> corner [1].
>>
>> I patched 5.4 alpha2 for review [2].
>>
>
> I think better idea would be to add another API function
> (zend_call_method_ex?), that implements arbitrary argument count (either via
> array or via varargs) and make zend_call_method be a case of it, maybe. This
> way you don't have to break existing API and also don't have to stop at 4.


Hi Stas,

That's what this patch does,  there is a new function
zend_call_method_multi, which takes an array, and I've revised zend_call
method to use it.  I could change it to varargs if that is preferred.

thanks,

-nathan

--0015175707e44c203704aff91cc5--