[patch] zend_call_method_with_N_params()
| From: | Masaki Fujimoto | Date: | Thu, 31 Jan 2008 14:58:51 +0000 |
| Subject: | [patch] zend_call_method_with_N_params() | ||
| Groups: | php.internals | ||
| Request: | Send a blank email to internals+get-35059@lists.php.net to get a copy of this message | ||
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>