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

Reply via email to