D10405: Stuff the "Couldn't find executable" message box into a queued lambda

[David Faure]

dfaure accepted this revision.
dfaure added inline comments.
This revision is now accepted and ready to land.

INLINE COMMENTS

> krun.cpp:1619
> +
> +            if (qGuiApp) {
> +                QTimer::singleShot(0, qGuiApp, [=]() {

Actually, KMessageBox requires qApp, not just qGuiApp.

(Then again, all of KRun requires widgets, so I'm not even sure when this could ever be null, currently)

REPOSITORY
  R241 KIO

BRANCH
  master

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

To: hein, dfaure, davidedmundson, mart, ngraham
Cc: #frameworks, michaelh
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20180215/c59b5707/attachment.html>