D25059: KPluginSelector: use new KAboutPluginDialog

Friedrich W. H. Kossebau noreply at phabricator.kde.org
Wed Oct 30 14:02:32 GMT 2019


kossebau added inline comments.

INLINE COMMENTS

> anthonyfieroni wrote in kpluginselector.cpp:799-803
> Can you guard for nullptr?

The pluginEntry? That would be inconsistent with the rest of the code here, which does no such checks.
We should be relatively safe here, as the slot should be only triggered if the "About" button was clicked, in which case the button has the focus, and such will focusedIndex() deliver a valid index.

So IMHO this code is working in promised conditions, if they are not met, something else is principally broken.

REPOSITORY
  R295 KCMUtils

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

To: kossebau, #frameworks, dfaure, apol
Cc: anthonyfieroni, kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20191030/d6ac8c62/attachment-0001.html>


More information about the Kde-frameworks-devel mailing list