mark florisson, 16.09.2011 23:33:
On 16 September 2011 22:29, Stefan Behnel wrote:
mark florisson, 16.09.2011 23:21:

On 16 September 2011 22:20, mark florisson wrote:

On 16 September 2011 22:03, Stefan Behnel wrote:

Robert Bradshaw, 15.09.2011 22:33:

See http://wiki.cython.org/ReleaseNotes-0.15.1 This is a bugfix only
release, we hope to get it out shortly.

I've added ticket 736 as a blocker, at least until I know what it takes
to fix it (looking into it right now).

http://trac.cython.org/cython_trac/ticket/736

Looking at the generated code I see

__pyx_cur_scope->__pyx_v_a = __pyx_k_1;
__pyx_cur_scope->__pyx_v_b = __pyx_k_2;

happening at unpacking time without a corresponding INCREF. Surely the
scope object should only ever own references and never borrow them?
The arguments from the args tuple are increffed however, just not the
default values.

Yes, I noticed that. See the updated bug description. I just wasn't sure
what the best fix was when I wrote the above.

Oh, sorry, I see.

No offence taken. It's just that I wrote basically all of that code myself, so I tend to know rather well where to put my fingers when something goes wrong. ;)

I think I found a good fix - and the code even ended up a tiny bit cleaner than it was before.

Stefan
_______________________________________________
cython-devel mailing list
cython-devel@python.org
http://mail.python.org/mailman/listinfo/cython-devel

Reply via email to