[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    Re: [kdelibs/frameworks] tier1/kwidgetsaddons/src: Introduce KMessageBoxNotifyInterface
From:       David Faure <faure () kde ! org>
Date:       2013-10-22 14:09:11
Message-ID: 8590440.WGf8CfTbFN () asterix
[Download RAW message or body]

On Tuesday 22 October 2013 15:13:23 Aurélien Gâteau wrote:
> I assumed the Qt plugin system did not delete its root object itself, but
> that  was wrong. See updated diff. Maybe we should also document the fact
> that he set*Interface() methods do not take ownership of the implementation
> objects?

Patch looks good.

Yes, please add documentation bit.

-- 
David Faure, faure@kde.org, http://www.davidfaure.fr
Working on KDE, in particular KDE Frameworks 5

[prev in list] [next in list] [prev in thread] [next in thread]