[Cython] Regression: cdef append() no longer works

2015-07-13 Thread Jeroen Demeyer

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?

___
cython-devel mailing list
cython-devel@python.org
https://mail.python.org/mailman/listinfo/cython-devel


Re: [Cython] Regression: cdef append() no longer works

2015-07-13 Thread Robert Bradshaw
On Mon, Jul 13, 2015 at 6:56 AM, Jeroen Demeyer  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/f75cad4297e437e9d9ab9e0ad43b8af6b351ba9a

> 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.
___
cython-devel mailing list
cython-devel@python.org
https://mail.python.org/mailman/listinfo/cython-devel


Re: [Cython] Regression: cdef append() no longer works

2015-07-13 Thread Stefan Behnel
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

___
cython-devel mailing list
cython-devel@python.org
https://mail.python.org/mailman/listinfo/cython-devel