> On Sept. 12, 2013, 7:39 a.m., David Faure wrote: > > staging/kservice/src/services/ktraderparsetree.cpp, line 715 > > <http://git.reviewboard.kde.org/r/112679/diff/1/?file=189053#file189053line715> > > > > kdelibs coding style -> no space inside (), '{' on the same line. > > > > foreach() would be more readable, too.
This would make the coding style of the whole file inconsistent. I suggest we keep the coding style at least file-consistent for now. (Until someone comes along and fixes the coding style there. It's all two-spaces-tabs and spaces inside parentheses. (Same above.) - Sebastian ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/112679/#review39864 ----------------------------------------------------------- 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
