<table><tr><td style="">rjvbb updated this revision to Diff 19654.<br />rjvbb marked 2 inline comments as done.<br />rjvbb edited the test plan for this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-4g6pjtbzkft2ueb/" rel="noreferrer">(Show Details)</a><br />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>This revision incorporates the suggested changes to AbstractFileManagerPlugin the project load timer and IProjectFileManager (instead of IProject).</p>

<p>I've made the dirwatcher checkbox visible only on Mac due to the apparent consensus that KDirWatch has negligible cost on Linux. The cost (on Mac) comes entirely from the single call to KDirWatch::addDir and is thus independent from any performance bugs in KDevelop's project loading; this justifies the proposition to defer the dirwatcher creation IMHO.</p></div></div><br /><div><strong>CHANGES TO TEST PLAN</strong><div><div style="white-space: pre-wrap; color: #74777D;">Developed in the 5.<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">1</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">2</span> branch on OS X 10.9 with Qt 5.8 and KF5 5.35.0; tested on Linux with the same versions. I'm not certain to what extent the underlying issue exists on Linux; my Linux system is simply not powerful enough to expect reasonable snappiness working with projects this big.<div style="padding: 8px 0;">...</div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R32 KDevelop</div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D7742?vs=19343&id=19654" rel="noreferrer">https://phabricator.kde.org/D7742?vs=19343&id=19654</a></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>AFFECTED FILES</strong><div><div>interfaces/iprojectcontroller.cpp<br />
interfaces/iprojectcontroller.h<br />
project/abstractfilemanagerplugin.cpp<br />
project/abstractfilemanagerplugin.h<br />
project/interfaces/iprojectfilemanager.h<br />
shell/projectcontroller.cpp<br />
shell/settings/projectconfig.kcfg<br />
shell/settings/projectpreferences.cpp<br />
shell/settings/projectpreferences.ui</div></div></div><br /><div><strong>To: </strong>rjvbb, KDevelop, mwolff, brauch<br /><strong>Cc: </strong>mwolff, kossebau, arrowdodger, brauch, zhigalin, kdevelop-devel, geetamc, Pilzschaf, akshaydeo, surgenight<br /></div>