Changes between Initial Version and Version 1 of Ticket #24285, comment 21


Ignore:
Timestamp:
11/30/17 09:16:18 (4 years ago)
Author:
mmezzarobba
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #24285, comment 21

    initial v1  
    66> Note that it used to be in `__init__` and not `__cinit__`. I does think that `__cinit__` should do the minimal job (memory allocation) and not set any value.
    77
    8 Yes, that's what I meant, sorry. As far as I can say, the general logic how how constructors are organized in the arb interface is that (i) `__cinit__()` does basic initialization, (ii) `__init__()` wraps the C-level intializers provided by arb (and should be reasonably fast), and (iii) `_element_constructor_()`, conversion morphisms and the like take care of everything else. This organization has pros and cons, but it sorts of make sense to me.
     8Yes, that's what I meant, sorry. As far as I can say, the general logic how how constructors are organized in the arb interface is that (i) `__cinit__()` does basic initialization, (ii) `__init__()` wraps the C-level intializers provided by arb (and should be reasonably fast), and (iii) `_element_constructor_()`, conversion morphisms and the like take care of everything else. This organization has pros and cons, but it sort of makes sense to me.