Re: Reflection API b0rked in HEAD and PHP_5_2
| From: | Hannes Magnusson | Date: | Fri, 09 Jun 2006 12:13:29 +0000 |
| Subject: | Re: Reflection API b0rked in HEAD and PHP_5_2 | ||
| References: | 1 | Groups: | php.internals |
| Request: | Send a blank email to internals+get-24031@lists.php.net to get a copy of this message | ||
Hi Sebastian
On 6/8/06, Sebastian Bergmann <sb@sebastian-bergmann.de> wrote:
http://phpfi.com/122573Looks like the arginfo patch broke it.., well, actually it uncovered a bug in ext/reflection. The messages are correct, ReflectionMethod::invokeArgs takes two arguments but ReflectionFunction::invokeArgs only takes one (ReflectionMethod extends ReflectionFunction) ergo=>E_STRICT in 5_2 and fatal in HEAD. The reason for it worked before is the engine has no idea about how many arguments each method takes - unless it gets that info specifically. We've got two options: don't provide the argument info (and therefor break our own OO rules) or take a deeper look into ext/reflection and fix the inherit problems.. -Hannes