#13192: some code clean up for sage/graphs/graph.py
----------------------------------+-----------------------------------------
       Reporter:  eisermbi        |         Owner:  jason, ncohen, rlm
           Type:  defect          |        Status:  needs_work        
       Priority:  major           |     Milestone:  sage-5.2          
      Component:  graph theory    |    Resolution:                    
       Keywords:  sparse6_string  |   Work issues:                    
Report Upstream:  N/A             |     Reviewers:  Nathann Cohen     
        Authors:  Birk Eisermann  |     Merged in:                    
   Dependencies:                  |      Stopgaps:                    
----------------------------------+-----------------------------------------

Comment (by kcrisman):

 No, removing whitespace right nearby isn't really a problem, but sometimes
 people ''are'' working on the same function (especially if it's a large
 one, like `plot` or something, so it's best to keep it relatively minimal.
 In this case, for instance, I could imagine someone working on fixing a
 type in the doc (say) where you removed whitespace, and then this might
 conflict.

 By the way, your commit message is good but maybe you could remove the
 `mq` part.

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/13192#comment:10>
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