<table><tr><td style="">rjvbb updated this revision to Diff 19945.<br />rjvbb edited the summary of this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-7ycuxctztjqfmnl/" 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/D7745" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>I'm reopening this. The patch still contains the code for import timing that I consider temporary and of use only during the review process. I'm aware it complexifies the main change, which isn't very complicated at all.</p>
<p>There may not be a significant benefit to deferring the start of the full project parse under normal circumstances on the overall import duration for all projects. The import times for the 2nd, 3rd etc. project do decrease for obvious reasons, meaning that the session is ready to go a bit sooner.</p>
<p>Deferring a resource-intensive feature also makes work on the efficiency of the preceding import easier.</p></div></div><br /><div><strong>CHANGES TO REVISION SUMMARY</strong><div><div style="white-space: pre-wrap; color: #74777D;"><span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">This is a companion patch to D7742 .<br />
<br />
</span>KDevelop currently starts the project parse run when the project has been imported. That's perfect when using single-project sessions, but it means that the import of each subsequent project has to compete with the parser. Project import is a process that can block the UI and should thus take as little time as possible.<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/D7745?vs=19328&id=19945" rel="noreferrer">https://phabricator.kde.org/D7745?vs=19328&id=19945</a></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D7745" rel="noreferrer">https://phabricator.kde.org/D7745</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>kdevplatform/shell/projectcontroller.cpp</div></div></div><br /><div><strong>To: </strong>rjvbb, KDevelop<br /><strong>Cc: </strong>mwolff, aaronpuchert, arrowdodger, brauch, kdevelop-devel, geetamc, Pilzschaf, akshaydeo, surgenight<br /></div>