<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/D7742" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>A few quick reactions, I'll get on to making (some of) the requested changes later this week or whenever I've migrated my build to the 5.2 branch.</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.e. a canonical "big" repository we can use for benchmarking.</p></blockquote>

<p>I propose the gcc 7.2.0 source tree, or for the really adventurous, the 6.x.0 and 7.y.0 source trees because that's what really exposed the issue to me.</p>

<p>Whether I'll be able to profile something that big on my Mac is another question, last time I tried to attach Instruments.app to a really busy KDevelop instance it just failed.</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>Just disabling this feature sounds like the wrong approach to me.</p></blockquote>

<p>What exactly is being watched, individual directories or individual files? The API suggests it might be the latter, and that suggests there must be a limit above which the feature becomes (too) expensive. Whatever that means exactly.</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>dito, but slightly changed - if the setting changed, did you recreate the watchers?</p></blockquote>

<p>No, I'm pretty sure at least the new watcher create function is only called just after the project's initial import.</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>remove or move into the Private class below, don't make it static</p></blockquote>

<p>Would you want to commit the timing feature? That was not my intention so I didn't want to bother making this a static Private class member (there shouldn't be any practical difference, AFAICT).</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>this should be a separate patch, it has nothing to do with dirwatches</p></blockquote>

<p>It was a separate patch until the dirwatch patch got too entwined with it. I propose I'll split off the parser component again when this is ready to be committed but I'd really prefer not having to keep them separate until that time.</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/D7742" rel="noreferrer">https://phabricator.kde.org/D7742</a></div></div><br /><div><strong>To: </strong>rjvbb, KDevelop, mwolff<br /><strong>Cc: </strong>mwolff, kossebau, arrowdodger, brauch, zhigalin, kdevelop-devel, geetamc, Pilzschaf, akshaydeo, surgenight<br /></div>