aacid added inline comments.

INLINE COMMENTS

> kbookmarkmanager.cpp:337
> +        if (d->m_dialogAllowed && qobject_cast<QApplication *>(qApp) && 
> QThread::currentThread() == qApp->thread()) {
> +            QMessageBox::critical(QApplication::activeWindow(), 
> QApplication::applicationName(), err);
> +        }

critical is a bit heavy in my opinion, i'd just go for warning for this and the 
debug line below.

> kbookmarkmanager.cpp:341
> +        qCritical() << QStringLiteral("Failed to start keditbookmarks");
> +        emit const_cast<KBookmarkManager *>(this)->error(err);
> +    }

Why const_cast?

REPOSITORY
  R294 KBookmarks

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

To: valeriymalov, #frameworks
Cc: aacid, ltoscano

Reply via email to