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