bruns accepted this revision.
bruns added inline comments.

INLINE COMMENTS

> davidedmundson wrote in kcmoduleqml.cpp:74
> Technically yes, but there's no sane scope for a code path where configModule 
> is emitting anything between deleting d and this.

Thanks for the confirmation, hope nobody finds an insane one ;-)

REPOSITORY
  R295 KCMUtils

BRANCH
  master

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

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

Reply via email to