Regression: cdef append() no longer works
Code like the following no longer works correctly: cdef class Foo: def __init__(self): self.append(1) cdef append(self, x): return x It seems that Cython now always assumes that append() is a special method: it incorrectly uses __Pyx_PyObject_Append. PS: it's not really clear if bug reports like these should go on this mailing list or on Trac (my impression is that Cython's Trac is not really used). What's the recommended procedure?
On Mon, Jul 13, 2015 at 6:56 AM, Jeroen Demeyer <jdemeyer@cage.ugent.be> wrote:
Code like the following no longer works correctly:
cdef class Foo: def __init__(self): self.append(1)
cdef append(self, x): return x
It seems that Cython now always assumes that append() is a special method: it incorrectly uses __Pyx_PyObject_Append.
Fixed: https://github.com/cython/cython/commit/f75cad4297e437e9d9ab9e0ad43b8af6b351...
PS: it's not really clear if bug reports like these should go on this mailing list or on Trac (my impression is that Cython's Trac is not really used). What's the recommended procedure?
Trac is certainly not as actively used as projects like Sage, though we do use it. For something simple like this I'd send a mail the list and then if it's not fixed right away make sure to file a bug so it doesn't get lost.
Robert Bradshaw schrieb am 14.07.2015 um 02:13:
On Mon, Jul 13, 2015 at 6:56 AM, Jeroen Demeyer wrote:
PS: it's not really clear if bug reports like these should go on this mailing list or on Trac (my impression is that Cython's Trac is not really used). What's the recommended procedure?
Trac is certainly not as actively used as projects like Sage, though we do use it. For something simple like this I'd send a mail the list and then if it's not fixed right away make sure to file a bug so it doesn't get lost.
+1, that works best for me, too. Stefan
participants (3)
-
Jeroen Demeyer -
Robert Bradshaw -
Stefan Behnel