[prev in list] [next in list] [prev in thread] [next in thread]
List: pykde
Subject: Re: [PyQt] SIP: memory leak in mapped type?
From: Giovanni Bajo <rasky () develer ! com>
Date: 2007-07-30 12:11:12
Message-ID: 46ADD560.8040803 () develer ! com
[Download RAW message or body]
On 7/28/2007 11:43 PM, Phil Thompson wrote:
>>> The leak that your test case demonstrated had nothing to do
>>> with /Transfer/ or
>>> sipTransferObj. If you still have a leak then I need another test
>>> case.
>> You mentioned that you fixed a memory leak in SIP with mapped types
>> and /Out/ annotations. This is one of the two leaks in my testcase (when
>> calling the function with the /Out/ annotation). But in the other leak
>> in my testcase, there is no /Out/ annotation, only a return value.
>>
>> Will that be fixed as well with tonight snapshot?
>
> Sorry, I misread your email. Both tests now use the same amount of memory.
>
> The other leak is down to you using sipConvertFromInstance() instead of
> sipConvertFromNewInstance(). I'd forgotten that a transfer object of NULL
> means different things to the two functions.
OK that's fine, I verified that the original problems are fixed with
latest snapshot and that fix.
I offer stl.sip for inclusion in SIP. It covers my own needs right now
(vectors) but shall I need to wrap more STL containers (sets, maps,
ecc.) I would updated it with support for them. I guess that it might be
useful for people wrapping C++ code with SIP.
There is a high degree for redundancy in it, as you can see. You might
want to think about it as a way to improve SIP itself.
--
Giovanni Bajo
_______________________________________________
PyQt mailing list PyQt@riverbankcomputing.com
http://www.riverbankcomputing.com/mailman/listinfo/pyqt
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic