[PATCH] Configure toolbars from RMB switch to the current toolbar

David Faure faure at kde.org
Wed Apr 28 23:21:15 BST 2004


On Wednesday 28 April 2004 22:15, Andras Mantia wrote:
> On Wednesday 28 April 2004 22:57, David Faure wrote:
> [...]
> > > The second version of the patch with the static member variable is
> > > attached.
> >
> > As I said, static objects in libraries are a big no no.
> > We've had enough crash due to them.
> 
> Even as member variables? Well, in this case const char is the way to
> go.

Yes. Sure. The fact that it's in a class is basically namespacing.
It's still _static_, i.e. its lifetime is the one of the library.

> 3rd patch attached.

Looks fine, thanks.

-- 
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