#8177: Breaking Integer's invariant can lead to a segfault in Sage 4.3.2.alpha1,
Mac OS X 10.6.2
-----------------------+----------------------------------------------------
   Reporter:  mvngu    |       Owner:  tbd       
       Type:  defect   |      Status:  needs_work
   Priority:  major    |   Milestone:  sage-4.3.3
  Component:  doctest  |    Keywords:            
     Author:           |    Upstream:  N/A       
   Reviewer:           |      Merged:            
Work_issues:           |  
-----------------------+----------------------------------------------------
Changes (by hivert):

  * author:  Florent Hivert =>


Comment:

 > No, it means that, with #8200, I have done my part at fixing my improper
 usage of ZZ. But the issue that such an improper usage can cause a
 segfault is still there, and I leave to the experts the decision of
 whether to fix it now, leave it to later, or resolve it as wontfix.
 >
 > I personally vote -1 for making it a wontfix.

 I strongly second Nicolas -1.

 Moreover, if the segfault is not removed the invariant
 {{{
     x.parent() == ZZ <==> x.class == Integer
 }}}
 must be clearly stated with a _big warning_ in the doc. My opinion is that
 the segfault must be left only if there is a very large performance
 penalty fixing it.
 By the way is this invariant an equivalence ? As far as I understood the
 segfault only came because we where breaking the {{{==>}}} part.

 Cheers,

 Florent

 By the way, I've removed myself as author since I won't be hacking in
 integers (my work in integrated in #8200).

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