#14794: DiGraph constructor doc describes `boundary` option wrong
---------------------------------------------------+------------------------
       Reporter:  mguaypaq                         |         Owner:  mguaypaq   
 
           Type:  defect                           |        Status:  
needs_review
       Priority:  major                            |     Milestone:  sage-5.12  
 
      Component:  graph theory                     |    Resolution:             
 
       Keywords:  graph, digraph, mutable default  |   Work issues:             
 
Report Upstream:  N/A                              |     Reviewers:             
 
        Authors:  Mathieu Guay-Paquet              |     Merged in:             
 
   Dependencies:                                   |      Stopgaps:             
 
---------------------------------------------------+------------------------

Comment (by ncohen):

 I never used this thing, and to me a code which is not documented might as
 well not be there, if it is only meant to be used by the guy who added it
 in the first place.

 This being said, has it become illegal to use list as an object's
 attribute ? I understand that you want an immutable version of everything
 that is in Sage, but if it means preventing people from writing code as
 they want to perhaps you should change your plans `O_o`

 By the way, who "frowns upon" mutable arguments ? It's useful to have a
 list somewhere, from time to time `O_o`

 Nathann

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/14794#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 unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/groups/opt_out.


Reply via email to