On 5 December 2012 17:11, mark florisson <markflorisso...@gmail.com> wrote: > On 5 December 2012 16:54, Yaroslav Halchenko <li...@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]markflorisso...@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]li...@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@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. _______________________________________________ cython-devel mailing list cython-devel@python.org http://mail.python.org/mailman/listinfo/cython-devel
