#15367: Empty lists while creating parents
-------------------------------------+-------------------------------------
Reporter: roed | Owner:
Type: defect | Status: new
Priority: major | Milestone: sage-5.13
Component: memleak | Resolution:
Keywords: | Merged in:
Authors: Nils Bruin | Reviewers: Simon King
Report Upstream: N/A | Work issues:
Branch: | Commit:
u/SimonKing/ticket/15367 | a2852e9610a76e6df392544b01b8e3c319b1cdd4
Dependencies: | Stopgaps:
-------------------------------------+-------------------------------------
Changes (by {'newvalue': u'Nils Bruin', 'oldvalue': ''}):
* commit: => a2852e9610a76e6df392544b01b8e3c319b1cdd4
* reviewer: => Simon King
* author: => Nils Bruin
Comment:
Replying to [comment:17 SimonKing]:
> Suggestion: I turn your two files into a git branch, remove the useless
stuff in coerce.pyx, and then we can start reviewing (and testing
performance).
Well, I simply did...
The first commit is the result of your drop-in replacement, the second
commit is removing the get_stats() method, which is a debug method that is
not functional any longer. I think the second commit is nothing but a
review patch. Filling Author and Reviewer fields accordingly---but the
review is not completed yet!
----
New commits:
||[[http://git.sagemath.org/sage.git/commit/?id=a2852e9|a2852e9]]||Remove
unused debug method CoercionModel_cache_maps.get_stats()||
||[[http://git.sagemath.org/sage.git/commit/?id=1724e0e|1724e0e]]||An
improved implementation of MonoDict and TripleDict||
--
Ticket URL: <http://trac.sagemath.org/ticket/15367#comment:19>
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.
