[Patch] Fix undeleted "static QFont *_font" in kglobalsettings.cpp

Matthias Kretz kretz at kde.org
Thu Dec 4 19:54:11 GMT 2008


On Thursday 04 December 2008 20:16:48 Thiago Macieira wrote:
> On Thursday 04 December 2008 17:46:47 Matthias Kretz wrote:
> > Thiago, do you have any good reason other than the name?
>
> Because we didn't mean it to be placed inside functions. It's called
> K_GLOBAL_STATIC, not K_LOCAL_STATIC.
>
> It works, sure, but we could decide to change it to something else, which
> may not work in a function location.

:) Then how about we add a K_LOCAL_STATIC macro that expands to 
K_GLOBAL_STATIC for now and be more strict wrt to the usage of the macros?

-- 
________________________________________________________
Matthias Kretz (Germany)                            <><
http://Vir.homelinux.org/





More information about the kde-core-devel mailing list