<table><tr><td style="">sandsmark created this revision.<br />sandsmark added reviewers: hindenburg, tcanabrava.<br />Herald added a project: Konsole.<br />Herald edited subscribers, added: konsole-devel; removed: Konsole.<br />sandsmark requested review of this revision.
</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/D26761">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p><a href="https://phabricator.kde.org/R319:59ff5d37301adf4753b590f42787055e6695b614" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">59ff5d37301adf4753b590f42787055e6695b614</a> completely ignored<br />
shouldUseNewProcess() which checks if we actually can re-use an existing<br />
process. This also broke --nofork, --separate etc.</p>

<p>See the comments in shouldUseNewProcess() for why and when Konsole<br />
actually needs to use a new process.</p>

<p>It also bypassed a crashfix because of the broken behavior of<br />
KDBusAddons because it didn't set the needToDeleteQApplication variable<br />
even though it forced reusing an existing an existing process.</p>

<p>See <a href="https://bugs.kde.org/show_bug.cgi?id=355545" class="remarkup-link" target="_blank" rel="noreferrer">https://bugs.kde.org/show_bug.cgi?id=355545</a></p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Konsole now reuses a process if possible and wanted by the user, otherwise forces a new process.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R319 Konsole</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D26761">https://phabricator.kde.org/D26761</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/main.cpp</div></div></div><br /><div><strong>To: </strong>sandsmark, hindenburg, tcanabrava<br /><strong>Cc: </strong>tcanabrava, hindenburg, konsole-devel, cent, EspadaV8, fbampaloukas, thsurrel, ngraham, maximilianocuria<br /></div>