<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="12" style="border: 1px #c9c399 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="https://git.reviewboard.kde.org/r/123675/">https://git.reviewboard.kde.org/r/123675/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On May 7th, 2015, 6:26 p.m. CEST, <b>Andreas Pakulat</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">I'd consider this a saner default and personally would likely remove the preference option and just change the behavior except for one thing: If you notice an instance is already started ask the user wether he wants to kill it or not. I run into a similar thing every once in a while with Eclipse starting an Eclipse application. Since that app uses a fixed workspace and a workspace can only ever be used by one instance of Eclipse I get a notification when I try to start the second instance. Most of the time I'm glad Eclipse doesn't just kill the already running instance to start a new one, since I simply forgot I still had it running.</p></pre>
</blockquote>
</blockquote>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Works for me, I'll drop the GUI option.</p></pre>
<br />
<p>- Aleix</p>
<br />
<p>On May 7th, 2015, 5:02 p.m. CEST, Aleix Pol Gonzalez wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="12" style="border: 1px #888a85 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
<tr>
<td>
<div>Review request for KDevelop.</div>
<div>By Aleix Pol Gonzalez.</div>
<p style="color: grey;"><i>Updated May 7, 2015, 5:02 p.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
kdevplatform
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">One of the things that bothered me the most when running applications on KDevelop is that eventually I'd had different instances of the program I'm working on at the same time, forcing me to kill them when I realized that.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">What this patch those is add a checkbox setting that lets the user specify whether she wants to have the separate process running.</p></pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Played with it using kdevelop.</p></pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>plugins/execute/nativeappconfig.cpp <span style="color: grey">(d5f4e83)</span></li>
<li>plugins/execute/iexecuteplugin.h <span style="color: grey">(c6b7fd7)</span></li>
<li>plugins/execute/executeplugin.cpp <span style="color: grey">(770e33e)</span></li>
<li>plugins/execute/executeplugin.h <span style="color: grey">(3e379b1)</span></li>
<li>plugins/execute/nativeappconfig.ui <span style="color: grey">(bb8b461)</span></li>
<li>plugins/execute/nativeappjob.h <span style="color: grey">(603f7b5)</span></li>
<li>plugins/execute/nativeappjob.cpp <span style="color: grey">(f8e2ba0)</span></li>
<li>project/builderjob.cpp <span style="color: grey">(826f348)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/123675/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>