#14711: Weak references in the coercion graph
-------------------------------------+-------------------------------------
       Reporter:  jpflori            |        Owner:  davidloeffler
           Type:  defect             |       Status:  needs_review
       Priority:  critical           |    Milestone:  sage-6.1
      Component:  number fields      |   Resolution:
       Keywords:  memleak, number    |    Merged in:
  field, QuadraticField              |    Reviewers:  Jean-Pierre Flori
        Authors:  Simon King         |  Work issues:
Report Upstream:  N/A                |       Commit:
         Branch:                     |  0af59ea93689cb6abb9d3fae0f1cf11f2aee5cca
  u/jpflori/ticket/14711             |     Stopgaps:
   Dependencies:                     |
-------------------------------------+-------------------------------------

Comment (by tscrim):

 My 2 cents. I don't think that `coerce_map_from()` should return the map
 which is used in the coercion system, but a copy of it. I believe this
 would be a better system since it's easily accessible to a user, so the
 user could mess with the map and the coercion system, and the result is
 used in error messages where the "copy" warning message has no purpose
 (for example the doctest in `ncsym/dual.py`). Subsequently I believe we
 should have a hidden function which returns the actual map, for those
 cases when one needs it.

 Also couldn't we use (some form of) #8878 get around some of the
 backtracking issues?

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