<div><div dir="auto">Point taken, I’ll rework the patch for a regexp of servicenane + ‘-‘ + pid, this will fix your second point.</div></div><div dir="auto">About your first point, do you think it’s a problem?</div><div dir="auto"><br></div><div dir="auto">Just speculating, if an app opens as multiple and then another instance tries to open as unique, it should reuse the existing one or merge?</div><div dir="auto"><br></div><div dir="auto">I did not try to change the registered name as actually nothing happens when a second instance of an unique app opens, it silently closes - so in a way I did not registered an unique application as name - pid, I just passed the arguments to the currently opened ::multiple application, and quit.</div><div dir="auto"><br></div><div dir="auto"><br></div><div><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, 3 Nov 2019 at 15:07 David Faure <<a href="mailto:noreply@phabricator.kde.org">noreply@phabricator.kde.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><table><tbody><tr><td>dfaure added a comment.
</td><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/D25084" target="_blank">View Revision</a></td></tr></tbody></table><br><div><div><p>If I read the patch correctly, then running a multi-instance app (<tt style="background:#ebebeb;font-size:13px">konsole-1234</tt>) and then running it as unique will lead to <tt style="background:#ebebeb;font-size:13px">konsole-1234</tt> instead of the expected <tt style="background:#ebebeb;font-size:13px">konsole</tt>.<br>
Worse, if there was a completely unrelated <tt style="background:#ebebeb;font-size:13px">konsolesettings</tt> application running, then konsole would try to register as <tt style="background:#ebebeb;font-size:13px">konsolesettings</tt> instead of <tt style="background:#ebebeb;font-size:13px">konsole</tt>.<br>
In other words, at least this should be <tt style="background:#ebebeb;font-size:13px">startsWith(d->serviceName + '-')</tt>, right?</p></div></div><br><div><strong>REPOSITORY</strong><div><div>R271 KDBusAddons</div></div></div><br><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D25084" target="_blank">https://phabricator.kde.org/D25084</a></div></div><br><div><strong>To: </strong>tcanabrava, adridg, dfaure<br><strong>Cc: </strong>dfaure, GB_2, davidedmundson, adridg, ngraham, kde-frameworks-devel, LeGast00n, michaelh, bruns<br></div></blockquote></div></div>