Newsgroups: php.internals Path: news.php.net Xref: news.php.net php.internals:24031 Return-Path: Mailing-List: contact internals-help@lists.php.net; run by ezmlm Delivered-To: mailing list internals@lists.php.net Received: (qmail 32334 invoked by uid 1010); 9 Jun 2006 12:13:34 -0000 Delivered-To: ezmlm-scan-internals@lists.php.net Delivered-To: ezmlm-internals@lists.php.net Received: (qmail 32318 invoked from network); 9 Jun 2006 12:13:34 -0000 Received: from unknown (HELO lists.php.net) (127.0.0.1) by localhost with SMTP; 9 Jun 2006 12:13:34 -0000 X-PHP-List-Original-Sender: hannes.magnusson@gmail.com X-Host-Fingerprint: 66.249.82.200 wx-out-0102.google.com Linux 2.4/2.6 Received: from ([66.249.82.200:58976] helo=wx-out-0102.google.com) by pb1.pair.com (ecelerity 2.0 beta r(6323M)) with SMTP id 56/93-30619-CE569844 for ; Fri, 09 Jun 2006 08:13:32 -0400 Received: by wx-out-0102.google.com with SMTP id s14so557180wxc for ; Fri, 09 Jun 2006 05:13:29 -0700 (PDT) DomainKey-Signature: a=rsa-sha1; q=dns; c=nofws; s=beta; d=gmail.com; h=received:message-id:date:from:to:subject:cc:in-reply-to:mime-version:content-type:content-transfer-encoding:content-disposition:references; b=jVTyWmm+QvOJDwQUcMYiiveSwFtw9WOaiqRb5CIN2W5RLz6fpZA/EPztESPfYxknpdvCMLcSaX1538FI0UA3cCDV8OIPGBkFjGaxpNvr8Gu+ReKnMqSd8bJ0YZPMJgTvsN5bWS5mI4ZYRx9KZK1vRxhLiIZ/ztVBoQ4pn9Rc3pA= Received: by 10.70.40.14 with SMTP id n14mr3370247wxn; Fri, 09 Jun 2006 05:13:29 -0700 (PDT) Received: by 10.70.91.5 with HTTP; Fri, 9 Jun 2006 05:13:29 -0700 (PDT) Message-ID: <7f3ed2c30606090513k3ebe9af8y3fcc251d09bd5911@mail.gmail.com> Date: Fri, 9 Jun 2006 14:13:29 +0200 To: "Sebastian Bergmann" Cc: internals@lists.php.net In-Reply-To: MIME-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Content-Disposition: inline References: Subject: Re: [PHP-DEV] Reflection API b0rked in HEAD and PHP_5_2 From: hannes.magnusson@gmail.com ("Hannes Magnusson") Hi Sebastian On 6/8/06, Sebastian Bergmann wrote: > http://phpfi.com/122573 Looks 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