Review Request 111686: KPluginFactory macros port
Sebastian Kügler
sebas at kde.org
Thu Jul 25 16:10:45 UTC 2013
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111686/
-----------------------------------------------------------
(Updated July 25, 2013, 4:10 p.m.)
Review request for KDE Frameworks and David Faure.
Description
-------
Make K_EXPORT_PLUGIN work with Qt's new plugin system
This patch changes the K_EXPORT_MACRO and the class it generates to be compatible with Qt's new plugin / metadata system. It basically replaces the old macros around q_plugin_instance with the new ones, using Q_INTERFACES. There's also a setter for the args, which are used to pass metadata into the plugin.
Otherwise, this is the minimal change, to make old plugin factories work atop the new framework.
This change is source-compatible, but the right .moc file when this macro is used from the .cpp file.
Diffs
-----
staging/kservice/src/plugin/kexportplugin.h cc5d58b
staging/kservice/src/plugin/kpluginfactory.h a5ea21b
staging/kservice/src/plugin/kpluginfactory.cpp 6bd2350
staging/kservice/src/plugin/kpluginfactory_p.h 09fcfe4
staging/kservice/src/plugin/kpluginloader.cpp 945c75b
Diff: http://git.reviewboard.kde.org/r/111686/diff/
Testing
-------
Loaded plugins using KService, KPluginLoader, QPluginLoader and Plasma::PluginLoader, all work as expected.
Thanks,
Sebastian Kügler
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20130725/5988b409/attachment.html>
More information about the Kde-frameworks-devel
mailing list