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