KUser -> kdecore

Simon Hausmann hausmann at kde.org
Fri Sep 19 09:40:44 BST 2003


On Thu, Sep 18, 2003 at 09:34:37PM -0400, Zack Rusin wrote:
> > I'd like to see this moved as well, as I use something similar in
> > ksplashml. Why does it use a pimpl? [1] As a general rule, I'd prefer
> > that we not use pimpls unless necessary as operator new can be
> > expensive.
> 
> 1) It's obviously necessary. (and It's the "d-pointer" in KDE, not 
> pimpl )
> 2) Everything can be expensive and new is definitely not even close to 
> being a problem when used for this purpose.

Plus 3) in this case it could be used to explicitly share the d
pointer and therefore make it less expensive as it means less new
calls (as the class has no mutators) .

Simon





More information about the kde-core-devel mailing list