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

Reply via email to