Plasma::Theme::self() -> default()

[Richard Dale]

On Wed, Apr 16, 2008 at 4:35 PM, Dan Meltzer <parallelgrapefruit at gmail.com>
wrote:

> On 4/16/08, Alexis Ménard <darktears31 at gmail.com> wrote:
> > yep i think ;)
> >
> >
> > On Wed, Apr 16, 2008 at 5:21 PM, Mike Evans <mike at asrc.info> wrote:
> > > dimsuz tried to make this change today, and since default is a C++
> > > keyword, gcc threw a fit.. would defaultTheme() be appropriate here
> > instead?
>
> Seems redundant...
>
> In Amarok we've switched to using a "the" namespace for singletons, I
> find it makes the code a bit more readable...
>
> Plasma::The::defaultTheme() instead of Plasma::Theme::defaultTheme()

Personally I think a 'The::' namespace is uglier. Anything to do with Themes
should be in the Theme namespace, otherwise surely you're breaking the idea
of namespaces?

-- Richard
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/panel-devel/attachments/20080416/d21f1b7a/attachment.html