2009/11/30 Jan Kundrát <jkt@gentoo.org>:
> Hassan Ibraheem wrote:
>> -    _optionList.setAutoDelete( true );
>
> Hi, I don't see any qDeleteAll() in the patch. I haven't looked at the
> code, but are you sure you do not leak memory here?
>
> Cheers,
> -jkt
>
> --
> cd /local/pub && more beer > /dev/mouth
>
>
I thought about that, and as far as I understood, we don't call any of
the Q3PtrList methods listed here about replacing setAutoDelete:
http://qt.nokia.com/doc/4.5/porting4.html#qptrlist-t
We don't remove/replace items from that list anywhere in the code.
However, maybe I should create a clear() method to be called before the
delete _annotationDialog;
in MainWindow/Window.cpp



-- 
Hassan Ibraheem