Re: [PHP-DEV] Revisit: zend_call_method() - n number of arguments

2011-10-23 Thread Nathan Nobbe
On Sun, Oct 23, 2011 at 12:50 AM, Stas Malyshev 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

Re: [PHP-DEV] Revisit: zend_call_method() - n number of arguments

2011-10-22 Thread Stas Malyshev
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 cou

[PHP-DEV] Revisit: zend_call_method() - n number of arguments

2011-10-22 Thread Nathan Nobbe
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]. Seems like the most appropriate time for a patch of this nature is during a major release, per the previous conversation. According to Marcus the patch is reasonable: