KDE/kdevplatform/shell

[Andreas Pakulat]

On 21.07.09 12:48:10, Milian Wolff wrote:
> Am Dienstag, 21. Juli 2009 07:23:09 schrieb Andreas Pakulat:
> > On 19.07.09 13:26:37, Milian Wolff wrote:
> > > SVN commit 999227 by mwolff:
> > >
> > > fix warning for copy constructor not initializing base class QObject.
> > >
> > > I first wanted to do : QObject(rhs) but that is private :-/ hope this fix
> > > is correct.
> >
> > We should never have a public/private/protected copy constructor for any
> > QObject based class. That doesn't make the slightest sence. For example
> > this one will loose the connection to the parent and hence will create a
> > memory leak.
> 
> so what should I do?

Well, look at other exported classes how they handle slots (I know we have
some other places to cleanup). For example projectcontroller.h in shell/.

Also checkout the C++ Binary Compat FAQ on techbase.kde.org:
http://techbase.kde.org/Policies/Library_Code_Policy
http://techbase.kde.org/Policies/Binary_Compatibility_Issues_With_C++

Andreas
 
-- 
You are number 6!  Who is number one?