<br><br><div class="gmail_quote">On Wed, Apr 16, 2008 at 4:35 PM, Dan Meltzer <<a href="mailto:parallelgrapefruit@gmail.com">parallelgrapefruit@gmail.com</a>> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div class="Ih2E3d">On 4/16/08, Alexis Ménard <<a href="mailto:darktears31@gmail.com">darktears31@gmail.com</a>> wrote:<br>
> yep i think ;)<br>
><br>
><br>
> On Wed, Apr 16, 2008 at 5:21 PM, Mike Evans <<a href="mailto:mike@asrc.info">mike@asrc.info</a>> wrote:<br>
> > dimsuz tried to make this change today, and since default is a C++<br>
> > keyword, gcc threw a fit.. would defaultTheme() be appropriate here<br>
> instead?<br>
<br>
</div>Seems redundant...<br>
<br>
In Amarok we've switched to using a "the" namespace for singletons, I<br>
find it makes the code a bit more readable...<br>
<br>
Plasma::The::defaultTheme() instead of Plasma::Theme::defaultTheme()</blockquote><div>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?<br>
<br>-- Richard<br> <br></div></div><br>