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

List:       kde-core-devel
Subject:    Re: KUser -> kdecore
From:       Simon Hausmann <hausmann () kde ! org>
Date:       2003-09-19 8:40:44
[Download RAW message or body]

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

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