[PATCH] KStaticDeleter -> K_GLOBAL_STATIC in kconfig_compiler
Thiago Macieira
thiago at kde.org
Mon May 21 10:23:12 BST 2007
Matthias Kretz said:
> Please review the attached patch. I'd like to commit today.
Looks good. Two comments, though (non-blocking):
1) the object creation is not thread-safe. Would that be a problem? I
don't know if you can use that kind of object from many threads.
2) any chance you can name the static variable something unique instead of
"s_self"? If you have to kconfig_compiled classes in the same project,
"s_self" would prevent enable-final from working.
--
Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org
PGP/GPG: 0x6EF45358; fingerprint:
E067 918B B660 DBD1 105C 966C 33F5 F005 6EF4 5358
More information about the kde-core-devel
mailing list