Hello Christian,

  patch looks fine and should go in.

  do you think ReflectionMethod::__construct implementation could be done
using parameterparsing 'f' rather than the spcial case 'o'? Unfortunately
we would still need the zs version unless we add another 'F' parameter
parsing that allows this two parameter case as well.

marcus

Friday, July 25, 2008, 5:41:49 PM, you wrote:

> Hi,

>> http://www.christian-seiler.de/temp/php/2008-07-24-reflection/reflection-closure-fixes-5.3.patch
>>  
>> 
>> http://www.christian-seiler.de/temp/php/2008-07-24-reflection/reflection-closure-fixes-6.patch
>>  

> The last CVS commit for apply_func_t and TSRMLS_CC conflicted with the
> patches, I merged the conflicting line manually and reuploaded the
> patches to the same location.

> Regards,
> Christian




Best regards,
 Marcus


-- 
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php

Reply via email to