[PATCH] Simplify plugin writing
David Faure
faure at kde.org
Mon Nov 5 17:22:07 GMT 2007
On Monday 05 November 2007, Will Stephenson wrote:
> When defining a plugin we use K_PLUGIN_FACTORY and K_EXPORT_PLUGIN macros to
> define its plugin factory and export it as a plugin. K_EXPORT_PLUGIN is
> defined in kpluginloader.h. I find #including this counterintuitive as a
> plugin author, and the compiler error when this is omitted had me mystified.
+2 :)
Same experience here.
> The attached patch simplifies plugin definition to just #include
> <kpluginfactory.h> and using the macros, by moving the definition of
> K_EXPORT_PLUGIN to a separate file, kexportplugin.h, which is included from
> both *loader and *factory headers.
Looks good.
> I have rebuilt kdelibs and kopete with
> this change and both build correctly; is anyone aware of more subtle use of
> plugins that it may break?
No but it would be good to update the unit test (klibloadertest4_module.cpp) to see if it
compiles without <kpluginfactory.h> now, as intended :)
--
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