<table><tr><td style="">rjvbb 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/D7995" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>But under what circumstances does <tt style="background: #ebebeb; font-size: 13px;">it adds a new folder or file item when none exists</tt>? Or rather, under what circumstances is that not being taken care of by the eventuallyReadFolder method? If the AbstractFileManagerPlugin creates new folders or files itself, why add them to the project view indirectly via the dirwatcher?</p>

<p>Is there a sequence of operations I can try in an actual session that you think should fail without that block of code (= in my current builds)?</p>

<p>(I assume you also re-read the KDirWatch doc?)</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>I honestly cannot understand your reasoning though. If you want to help out with KDevelop development, you have to target master.</p></blockquote>

<p>I hope then that this is still small enough to make 5.2.1 . I don't just help out with KDevelop development, I use it, and the master branch has too often been breaking things for me. There's no problem as long as I don't have to maintain 2 very different versions of the same patch.</p>

<p>What about the direct commit of the restartDirScan() check, can that one be merged immediately into 5.2 (or committed there too)?</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">auto watcher = m_watchers.value(folder->project());</pre></div></blockquote>

<p>I've been meaning to ask, what's the advantage of using auto here? I find it doesn't help reading the code and it feels like going back to the sloppy-typed days of good ole C.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R32 KDevelop</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D7995" rel="noreferrer">https://phabricator.kde.org/D7995</a></div></div><br /><div><strong>To: </strong>rjvbb, KDevelop, mwolff<br /><strong>Cc: </strong>arrowdodger, kfunk, dfaure, mwolff, brauch, kdevelop-devel, geetamc, Pilzschaf, akshaydeo, surgenight<br /></div>