#13501: Bugs in C3'salgorithm implementation in sage.misc.c3
-------------------------------------------+--------------------------------
       Reporter:  nthiery                  |         Owner:  nthiery     
           Type:  defect                   |        Status:  needs_review
       Priority:  major                    |     Milestone:  sage-5.4    
      Component:  categories               |    Resolution:              
       Keywords:  method resolution order  |   Work issues:              
Report Upstream:  N/A                      |     Reviewers:  Simon King  
        Authors:  Nicolas M. ThiƩry        |     Merged in:              
   Dependencies:  #12895                   |      Stopgaps:              
-------------------------------------------+--------------------------------

Comment (by nthiery):

 Replying to [comment:8 SimonKing]:
 > Another optimization:
 > {{{
 > for j from 0 <= j < nbheads:
 >     if j == i:
 >         continue
 >     if O in tailsets[j]:
 >         next_item_found = False
 > if next_item_found:
 >     ...
 > }}}
 > Apparently, after the line `next_item_found = False`, one should insert
 a `break`, because otherwise the loop is finished to no avail.

 Good point; I am trying it right now!

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