meven added inline comments.
INLINE COMMENTS
> kpluginselector.cpp:855
> + }
> + KCModuleProxy *currentModuleProxy = new
> KCModuleProxy(moduleInfo, moduleProxyParentWidget, arguments);
> if (currentModuleProxy->realModule()) {
Adding a fileName field to KCModuleProxy would make more sense to me, and do it
by default.
Plus KCModuleProxy has already access to the fileName since it receives
moduleInfo.
But I am not a specialist.
REPOSITORY
R295 KCMUtils
REVISION DETAIL
https://phabricator.kde.org/D29201
To: alex, #plasma, ngraham, meven, broulik, mart
Cc: mart, apol, kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham,
bruns