Review Request 117049: Ensure that pluginVersion gets resolved for multiple KPluginLoader

Martin Gräßlin mgraesslin at kde.org
Wed Mar 26 14:35:23 UTC 2014


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

(Updated March 26, 2014, 3:35 p.m.)


Review request for KDE Frameworks and David Faure.


Changes
-------

integrated into the new unit tests


Repository: kservice


Description
-------

Ensure that pluginVersion gets resolved for multiple KPluginLoader

If the library was already loaded the KPluginLoader did not resolve the
pluginVersion, so any check for the pluginVersion failed.


Diffs (updated)
-----

  autotests/kpluginloadertest.cpp d8bae4ea6c5295f4648c38a79d871ba7649695f1 
  src/plugin/kpluginloader.cpp 8c34fc7e903a6e0fbdc239d53980e5b861d8a52a 

Diff: https://git.reviewboard.kde.org/r/117049/diff/


Testing
-------

see included unit test


Thanks,

Martin Gräßlin

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20140326/3cb39715/attachment.html>


More information about the Kde-frameworks-devel mailing list