----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/112679/#review39859 -----------------------------------------------------------
2 issues I'd like to get advise for inline. staging/kservice/src/services/ktraderparsetree_p.h <http://git.reviewboard.kde.org/r/112679/#comment29389> Here, the KPluginInfo::List goes out of scope immediately, so we end up with an invalid reference. It's not accessed, but this seems rather ugly. It also yields a compiler warning. How can this be solved elegantly? (Same problem in the other ctor, where the KService::List& becomes invalid.) staging/kservice/src/services/ktraderparsetree_p.h <http://git.reviewboard.kde.org/r/112679/#comment29390> here, the KService::List& becomes invalid, see issue right above. - Sebastian Kügler On Sept. 12, 2013, 1:33 a.m., Sebastian Kügler wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/112679/ > ----------------------------------------------------------- > > (Updated Sept. 12, 2013, 1:33 a.m.) > > > Review request for KDE Frameworks and David Faure. > > > Description > ------- > > This patch allows search through KPluginInfo::List, additionally to > KService::List. Together with the other patches from my set It makes it > possible to search through the plugin's (JSON-based) metadata, using our > constraint syntax. > > It adds a ctor that takes a KPluginInfo::List& and adds the necessary > branches to look within either the KService::List or the KPluginInfo::List. > > There's one problem I don't know an elegant solution to, will point that out > in the diff. > > > This, and the other patches from this set are also in > kdelibs[sebas/pluginlocator] > > > Diffs > ----- > > staging/kservice/src/services/ktraderparsetree.cpp 4257e43 > staging/kservice/src/services/ktraderparsetree_p.h 7664943 > > Diff: http://git.reviewboard.kde.org/r/112679/diff/ > > > Testing > ------- > > - Added tests (separate review), all pass > - Ported Plasma::PluginLoader from KServiceTypeTrader to KPluginTrader and > tested the querying in a real-world scenario, no regressions spotted > - No regressions in existing tests > > > Thanks, > > Sebastian Kügler > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
