D13034: Add mechanism to notify other clients of config changes over DBus

Vlad Zagorodniy noreply at phabricator.kde.org
Tue May 22 13:41:52 UTC 2018


zzag added inline comments.

INLINE COMMENTS

> kconfig.cpp:526
> +    QDBusConnection::sessionBus().send(message);
> +#endif
> +}

#else
  Q_UNUSED(changes);
  Q_UNUSED(path);
  #endif

> kconfigwatcher.cpp:82
> +        KConfigGroup group = d->m_config->group(QString());//top level group
> +        for(const QString &groupName: it.key().split(QLatin1Char('\x1d'))) { //magic char, see KConfig
> +            group = group.group(groupName);

Won't this detach?

REPOSITORY
  R237 KConfig

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

To: davidedmundson
Cc: zzag, kde-frameworks-devel, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20180522/1d67b012/attachment-0001.html>


More information about the Kde-frameworks-devel mailing list