Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:35059
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: unknown (pb1.pair.com: domain php.net does not designate 219.94.145.87 as permitted sender)
Message-ID: <47A1E22B.6020200@php.net>
Date: Thu, 31 Jan 2008 23:58:51 +0900
User-Agent: Thunderbird 2.0.0.9 (Macintosh/20071031)
MIME-Version: 1.0
To: helly@php.net
Cc: internals@lists.php.net
Content-Type: text/plain; charset=ISO-2022-JP
Content-Transfer-Encoding: 7bit
Subject: [patch] zend_call_method_with_N_params()
From: fujimoto@php.net (Masaki Fujimoto)

hello marcus,

after a long interval, I've restarted to hack PHP and ZE again, and I
found that zend_call_method could take only 2 args at most...(I just
wanted to call methods w/ 3 or more args in my extensions:)

I think we can easily rewrite this w/ va_list, and since most of the
extensions are call zend_call_method() via zend_call_method_with* macro
so that no BC breaking things will happen.

# AFAIK only spl use zend_call_method() directly and perhaps we can
easily sync w/ updated interfaces (TSRM fixes will do)

pathces are here:
[PHP_5_2 / PHP_5_3]
http://labs.gree.jp/data/patch/zend_interfaces_var_list.php_5_2.patch

[HEAD]
http://labs.gree.jp/data/patch/zend_interfaces_var_list.head.patch

# C99 feature (__VA_ARGS__) is used in my patch for HEAD...(is it
acceptable?)

if this is ok, it's much appreciated if you apply these patches. thank you.

-- 
Masaki Fujimoto
<fujimoto@php.net>