Dolphin crash fix

Rafael Fernández López ereslibre at kde.org
Sun Dec 9 13:38:44 GMT 2007


Hi Peter,

> I've committed 2 patches yesterday which fixed the same issue. In the first
> patch I also used deleteLater(), but David pointed out that it gets tricky
> to find out memory leaks by using deleteLater() in the shutdown process. So
> in the second patch I used 'delete' again, but assured that the directory
> lister disconnects all signals in the destructor of the view-container. I
> think this works now very well... As in my patch also another memory leak
> is fixed (I did not release the proxy model), I'd propose that we leave it
> as it is. Is this OK for you?
>
> I'm sorry that we worked on the same issue, I hope you did not spend too
> much time on this issue :-/

Ah no problem ! That happens because of no updating ! hehe, no, fortunately 
this one was trivial from the backtrace :)


Bye and thanks,
Rafael Fernández López

GPG Fingerprint: B9F4 4730 43F8 FFDD CC5E BA8E 724E 406E 3F01 D070
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part.
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20071209/ed3f487b/attachment.sig>


More information about the kfm-devel mailing list