#12894: Add classcall setter in ClasscallMetaclass
--------------------------------------------------+-------------------------
       Reporter:  nthiery                         |         Owner:  jason       
  
           Type:  enhancement                     |        Status:  
needs_review  
       Priority:  major                           |     Milestone:  sage-5.1    
  
      Component:  misc                            |    Resolution:              
  
       Keywords:  classcall UniqueRepresentation  |   Work issues:              
  
Report Upstream:  N/A                             |     Reviewers:  Florent 
Hivert
        Authors:  Nicolas M. ThiƩry               |     Merged in:              
  
   Dependencies:                                  |      Stopgaps:              
  
--------------------------------------------------+-------------------------

Comment (by hivert):

 Hi Nicolas,

 Thanks for this one. I've a request: You should describe the input of
 {{{_set_classcall__}}} saying that the function must accept the class
 itself
 at first argument. I'll try to post a review patch.

 I'm also w little worried about the interface: Alternatively we could
 either

 - make the attribute {{{classcall}}} public (maybe by renaming it with
   underscores (I mistyped underscare at first ;-)).

 - or use a property to have a real setter allowing to a nicer syntax. I'm
   pretty sure Cython allows ot too.

 Florent

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/12894#comment:3>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica, 
and MATLAB

-- 
You received this message because you are subscribed to the Google Groups 
"sage-trac" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/sage-trac?hl=en.

Reply via email to