Re: [sage-devel] Re: cython string problem

2016-01-11 Thread Daniel Krenn
On 2016-01-10 20:36, Volker Braun wrote: > 1) don't cdef class attributes, it just makes debugging unnecessary > hard. Unless you are wrapping C-level types where you can't avoid it, of > course. Just keep it in python, maybe use cpdef if you must. Ok. > 2) I'd recommend using __cinit__ instead o

[sage-devel] Re: cython string problem

2016-01-10 Thread Volker Braun
1) don't cdef class attributes, it just makes debugging unnecessary hard. Unless you are wrapping C-level types where you can't avoid it, of course. Just keep it in python, maybe use cpdef if you must. 2) I'd recommend using __cinit__ instead of __init__ with cdef classes and use more of a RAII