On Apr 30, 2009, at 9:36 AM, Stefan Behnel wrote:

> Stefan Behnel wrote:
>> Stefan Behnel wrote:
>>> Robert Bradshaw wrote:
>>>> http://hg.cython.org/cython-closures/file/c4ee199f08a2/tests/run/ 
>>>> closures_T82.pyx
>>>>
>>>> Before you get your hopes up too much, I want to add a big
>>>> disclaimer that though I tried to be careful, this is very first-
>>>> draft code and there are bound to be issues (e.g. I think I've run
>>>> into some with respect to Python arguments).
>>> I'll give it a try during the next days.
>>
>> I exercised it a bit and added more test cases. That revealed a  
>> couple of
>> problems, although I must say that it works pretty well in general.
>>
>> I added comments to the tests as far as I got with analysing them.  
>> One
>> segfaults pretty badly, which makes it harder to figure out.
>
> Ok, that was due to ref-counting problems.

I was hoping that's all it was (not that tracking down ref-counting  
issues is always easy...)

> All of these work for me now:
>
> http://hg.cython.org/cython-closures/file/626e754738b6/tests/run/ 
> closures_T82.pyx
>
> I find that pretty impressive. Very good work, Robert!

Thanks.

> This looks like it could go right into 0.12 rather than waiting for  
> 0.13.

Yep, I think that's very feasible too.

- Robert

_______________________________________________
Cython-dev mailing list
[email protected]
http://codespeak.net/mailman/listinfo/cython-dev

Reply via email to