dfaure accepted this revision.
dfaure added inline comments.
This revision is now accepted and ready to land.

INLINE COMMENTS

> kconfigwatcher.cpp:72
> +    watchedPaths <<QStringLiteral("/") + d->m_config->name();
> +    for (const QString file: d->m_config->additionalConfigSources()) {
> +        watchedPaths << QStringLiteral("/") + file;

const QString &

and additionalConfigSources should be put into a local const var first, to 
avoid a detach.

REPOSITORY
  R237 KConfig

BRANCH
  master

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

To: davidedmundson, broulik, dfaure
Cc: dfaure, broulik, zzag, kde-frameworks-devel, michaelh, ngraham, bruns

Reply via email to