[PATCH] kdelibs: Memory leaks (d-pointer not deleted) fixed

David Faure faure at kde.org
Thu Oct 28 11:04:05 BST 2004


On Thursday 28 October 2004 10:39, Andre Eisenbach wrote:
> Hi!
> 
> Yet another patch for problems detected from the automated test script
> posted on dot.kde.org.
> 
> This patch adds code to delete d-pointers used in various classes
> inside kdelibs.

Warning!!! Do this only when the constructor does "d = new ..."

The KProtocolManager change is wrong for sure: "d" is a static pointer,
deleted by a KStaticDeleter. This patch would lead to double-deletion.

-- 
David Faure, faure at kde.org, sponsored by Trolltech to work on KDE,
Konqueror (http://www.konqueror.org), and KOffice (http://www.koffice.org).




More information about the kde-core-devel mailing list