#9880: Segfault in PyNaC 0.2.0.p4
-------------------------+--------------------------------------------------
   Reporter:  jpflori    |          Owner:  burcin    
       Type:  defect     |         Status:  needs_work
   Priority:  major      |      Milestone:  sage-4.7.2
  Component:  symbolics  |       Keywords:  pynac     
Work_issues:             |       Upstream:  N/A       
   Reviewer:             |         Author:            
     Merged:             |   Dependencies:            
-------------------------+--------------------------------------------------

Comment(by jpflori):

 The latest version of the patch giving access to the internal printing
 functions, gives the operator() instead of the compare(...) functions.

 I'm not sure this was intended, even if it is completely ok for the
 randomized testing.

 Anyway the line using it is cmp(..) == 1, so going back to the previous
 semantic is also ok.

 Hence I've provided a modified patch to do it.

 I'll also reversed the arguments in _cmp_c_impl and get rid of the minus
 sign.

 I do not see any reason for this choice, but if there was, please revert
 my change.

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