[Cython] (no subject)

[mark florisson]

On 5 December 2012 17:11, mark florisson <markflorisson88 at gmail.com> wrote:
> On 5 December 2012 16:54, Yaroslav Halchenko <lists at onerussian.com> wrote:
>> Thank you Bradley,
>>
>> This was easy enough even so that I could fix it (I think) -- just a
>> typo:
>>
>>     https://github.com/cython/cython/pull/165
>>
>> feedback would be welcome ;)
>>
>> On Mon, 03 Dec 2012, Bradley M. Froehle wrote:
>>
>>>    I ran `git bisect`:
>>>    d96dfdbb290d23bf3b4a186dc5b1b5d9ee7fcaa5 is the first bad commit
>>>    commit d96dfdbb290d23bf3b4a186dc5b1b5d9ee7fcaa5
>>>    Author: Mark Florisson <[1]markflorisson88 at gmail.com>
>>>    Date: � Tue Apr 10 15:01:00 2012 +0100
>>>    � � Decref memoryview slice class attributes
>>>    [2]https://github.com/cython/cython/commit/d96dfdbb290d23bf3b4a186dc5b1b5d9ee7fcaa5
>>>    -Brad
>>>    On Mon, Dec 3, 2012 at 1:10 PM, Yaroslav Halchenko
>>>    <[3]lists at onerussian.com> wrote:
>>
>>>      reproduced with cython 0.17.2 (+ few post release fixes), originally
>>>      detected/reported [1] with �0.17.1 on Debian systems using debug build
>>>      of
>>>      Python:
>>
>>>      $> python-dbg -c 'import pyximport as pi; pi.install(); import weakfail;
>>>      s=weakfail.foo(42)'
>>>      python-dbg: ../Modules/gcmodule.c:366: visit_decref: Assertion
>>>      `gc->gc.gc_refs != 0' failed.
>>>      [1] � �12648 abort � � �python-dbg -c
>>
>>>      $> cat weakfail.pyx
>>>      import weakref
>>>      foo_dict = weakref.WeakValueDictionary()
>>
>>>      cdef class Foo:
>>>      � � cdef object __weakref__
>>
>>>      def foo(key):
>>>      � � obj = Foo()
>>>      � � foo_dict[key] = obj
>>>      � � return obj
>>
>>>      it seems to work fine with cython 0.15.2 . �Any ideas? �requires
>>>      separate/new bug report?
>>
>>>      [1] [4]http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=692313
>> --
>> Yaroslav O. Halchenko
>> Postdoctoral Fellow,   Department of Psychological and Brain Sciences
>> Dartmouth College, 419 Moore Hall, Hinman Box 6207, Hanover, NH 03755
>> Phone: +1 (603) 646-9834                       Fax: +1 (603) 646-1419
>> WWW:   http://www.linkedin.com/in/yarik
>> _______________________________________________
>> cython-devel mailing list
>> cython-devel at python.org
>> http://mail.python.org/mailman/listinfo/cython-devel
>
> Thanks, I merged it.

It'd be nice if you could add this as a testcase in the tests/run
directory as a "don't segfault" kind of test.