<table><tr><td style="">dfaure requested changes to this revision.<br />dfaure added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D8043" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>In addition to being BIC, this is not the way to add thread safety. Either the class should be threadsafe (and then it should use mutexes internally) or it doesn't need to be threadsafe (and that can just be documented).<br />
Why not create one KDirWatch per thread?</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R244 KCoreAddons</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D8043" rel="noreferrer">https://phabricator.kde.org/D8043</a></div></div><br /><div><strong>To: </strong>rjvbb, Frameworks, mwolff, dfaure<br /><strong>Cc: </strong>dfaure, mwolff, kde-frameworks-devel<br /></div>