dfaure added a comment.
If I read the patch correctly, then running a multi-instance app (`konsole-1234`) and then running it as unique will lead to `konsole-1234` instead of the expected `konsole`. Worse, if there was a completely unrelated `konsolesettings` application running, then konsole would try to register as `konsolesettings` instead of `konsole`. In other words, at least this should be `startsWith(d->serviceName + '-')`, right? REPOSITORY R271 KDBusAddons REVISION DETAIL https://phabricator.kde.org/D25084 To: tcanabrava, adridg, dfaure Cc: dfaure, GB_2, davidedmundson, adridg, ngraham, kde-frameworks-devel, LeGast00n, michaelh, bruns
