<table><tr><td style="">dfaure 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/D18089">View Revision</a></tr></table><br /><div><div><p>Looks good. This assumes that no process exits cleanly (code 0) while not showing anything to the user, but even if that happened, better no klauncher error in that case, than an error in a normal case like the kate one.</p></div></div><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D18089#inline-99710">View Inline</a><span style="color: #4b4d51; font-weight: bold;">klauncher.cpp:340</span></div>
<div style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; white-space: pre-wrap; clear: both; padding: 4px 0; margin: 0;"><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">                <span class="n">request</span><span style="color: #aa2211">-></span><span class="n">status</span> <span style="color: #aa2211">=</span> <span class="n">KLaunchRequest</span><span style="color: #aa2211">::</span><span class="n">Done</span><span class="p">;</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">                <span class="n">qCDebug</span><span class="p">(</span><span class="n">KLAUNCHER</span><span class="p">)</span> <span style="color: #aa2211"><<</span> <span class="n">pid</span> <span style="color: #aa2211"><<</span> <span style="color: #766510">"died with exitStatus= 0, requestDone. status="</span> <span style="color: #aa2211"><<</span> <span class="n">request</span><span style="color: #aa2211">-></span><span class="n">status</span> <span class="p">;</span>
</div><div style="padding: 0 8px; margin: 0 4px; ">            <span class="p">}</span> <span style="color: #aa4000">else</span> <span class="p">{</span>
</div></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">To make this less scary (since it's not an error) I would suggest</p>

<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);"><< "exited without error, request done. status=" <<</pre></div></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R303 KInit</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D18089">https://phabricator.kde.org/D18089</a></div></div><br /><div><strong>To: </strong>ahmadsamir, dfaure<br /><strong>Cc: </strong>kde-frameworks-devel, michaelh, ngraham, bruns<br /></div>