Review Request 111688: QVariantList-based ctor for KPluginInfo

Sebastian Kügler sebas at kde.org
Mon Jul 29 00:25:30 UTC 2013


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111688/
-----------------------------------------------------------

(Updated July 29, 2013, 12:25 a.m.)


Review request for KDE Frameworks and David Faure.


Changes
-------

Fixed all issues pointed out, except the one about the format of the arriving metadata.


Description
-------

This patch allows KPluginInfo to be created from a QVariantList, as it is typically passed in from KPluginFactory (using the patch which changes the macro there).


Diffs (updated)
-----

  staging/kservice/src/services/kplugininfo.h c000008 
  staging/kservice/src/services/kplugininfo.cpp bd1eaec 

Diff: http://git.reviewboard.kde.org/r/111688/diff/


Testing
-------

Works with plugins built in the right way: A valid KPluginInfo is created from the plugin's metadata.


Thanks,

Sebastian Kügler

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20130729/14e8420b/attachment.html>


More information about the Kde-frameworks-devel mailing list