#4096: [with patches, needs review] pari precision interface
------------------------------+---------------------------------------------
Reporter: cremona | Owner: cremona
Type: defect | Status: new
Priority: major | Milestone: sage-3.1.3
Component: basic arithmetic | Resolution:
Keywords: |
------------------------------+---------------------------------------------
Comment (by mabshoff):
Hmm, I don't like the following change:
{{{
178 s = str(self)
179 428 import sage.libs.pari.gen_py
180 return sage.libs.pari.gen_py.pari, (s,)
429 return sage.libs.pari.gen_py.pari, (str(self),)
}}}
I am not 100% certain, but if s were a C type object the above would cause
a leak. I have fixed similar issues over and over again in code all over
Sage and I suspect that the reference count for "return str(foo)" might be
broken somehow. I have zero prove of this, obviously, but I intent to dig
deep one day.
Cheers,
Michael
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/4096#comment:6>
Sage <http://sagemath.org/>
Sage - Open Source Mathematical Software: Building the Car Instead of
Reinventing the Wheel
--~--~---------~--~----~------------~-------~--~----~
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
-~----------~----~----~----~------~----~------~--~---
