[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