broulik added inline comments.

INLINE COMMENTS

> davidedmundson wrote in pluginloader.cpp:195
> Seems the wrong approach we know the ID, there's no point iterating in the 
> first place.
> 
> https://api.kde.org/frameworks/kcoreaddons/html/classKPluginLoader.html#a3a69c4f1826926a2145f14770b8b5010

We know the ID but not the file name of the library, so `findPlugin` won't work.

REPOSITORY
  R242 Plasma Framework (Library)

REVISION DETAIL
  https://phabricator.kde.org/D14009

To: broulik, #plasma, davidedmundson, mart
Cc: ngraham, zzag, kde-frameworks-devel, michaelh, bruns

Reply via email to