----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/112660/#review39935 -----------------------------------------------------------
Ship it! ... after fixing the last two small issues. staging/kservice/src/services/kplugininfo.h <http://git.reviewboard.kde.org/r/112660/#comment29479> missing docu for libraryPath staging/kservice/src/services/kplugininfo.h <http://git.reviewboard.kde.org/r/112660/#comment29480> can be removed now, no? - David Faure On Sept. 12, 2013, 7:07 p.m., Sebastian Kügler wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/112660/ > ----------------------------------------------------------- > > (Updated Sept. 12, 2013, 7:07 p.m.) > > > Review request for KDE Frameworks and David Faure. > > > Description > ------- > > This patch prepares KPluginInfo for usage with KPluginTrader (to be submitted > in a separate patch). It basically makes KPluginInfo's API a little bit more > like KService by adding a property(QString) accessor to the plugin info. This > allows us on the one hand > > Part of this patch, and much of its churn, is the internal change from > independent QStrings and QStringLists to a QVariantMap. This is how the > metadata comes in from KPlugin*, and it allows us to make properties > accessible by name. > > There's also a fair bit of moving from QLatin1String to QStringLiteral in > there, most of these lines needed changes anyway. Additionally, the keys of > properties are now shared in the d-pointer. > > This change is source compatible to the old version. > > > Diffs > ----- > > staging/kservice/src/services/kplugininfo.h c2e5bab > staging/kservice/src/services/kplugininfo.cpp 21e0882 > > Diff: http://git.reviewboard.kde.org/r/112660/diff/ > > > Testing > ------- > > All tests still pass, no regressions encountered otherwise. > > > Thanks, > > Sebastian Kügler > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
