#14159: Don't install callbacks on values of TripleDict, MonoDict
------------------------------+---------------------------------------------
Reporter: nbruin | Owner: tbd
Type: defect | Status: needs_review
Priority: major | Milestone: sage-5.8
Component: memleak | Resolution:
Keywords: | Work issues:
Report Upstream: N/A | Reviewers: Nils Bruin
Authors: Simon King | Merged in:
Dependencies: #13387 | Stopgaps:
------------------------------+---------------------------------------------
Comment (by SimonKing):
Replying to [comment:41 nbruin]:
> Replying to [comment:40 SimonKing]:
> > But at least in cython modules, when one uses cdef'd variables of type
`MonoDict` or `TripleDict`, using a cdef or cpdef set/get method directly
should be faster than calling `__set/getitem__`, isn't it?
>
> Only because it's using the `cdef` part then, I think.
"Only" or not: If it is faster then it should be used.
And I don't think it is "only" cdef. It is also the `set/get` expects a
fixed number of arguments, while in `__set/getitem__` the given arguments
are put into a tuple, need to be unpacked, the length of the tuple must be
checked. So, set/get could be faster anyway.
I am now testing---as soon as half of the Sage library is rebuilt. I hate
patches touching coerce_dict.pxd!!!
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/14159#comment:42>
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?hl=en.
For more options, visit https://groups.google.com/groups/opt_out.
