On Wed, Mar 20, 2013 at 8:55 PM, Jörg Ehrichs <span dir="ltr"><<a href="mailto:Joerg.Ehrichs@gmx.de" target="_blank">Joerg.Ehrichs@gmx.de</a>></span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div class="im">> IMHO all of these need to be replaced with the async variants. This</div><div class="im">
> unfortunately affects a lot of your code. The service job should be<br>
> easy as you don't wait for the results, the service engine is less of<br>
> any easy change. You might want to get a second opinion before<br>
> starting.<br>
><br>
<br>
</div>I was hoping that it could stay this way (less code this way around)<br>
but ideally you are right, this should go async in case one of the<br>
services hangs in such a method (doubt that happens, but<br>
we never know).<br>
<br>
But I'll wait for 1-2 more opinions on this first.<br></blockquote><div><br></div><div>Fwiw synchronous calls are never good. In KDE Telepathy we were also using sync dbus calls and this caused for example Amarok hanging for few seconds on startup (had to do with mpris detection); and these are super hard to debug/find. Also as plasma-desktop represents the main shell UI, I think you never ever want to block that, even if just for a second.</div>

<div> </div><div>Cheers</div></div>-- <br><div><span style="color:rgb(102,102,102)">Martin Klapetek | KDE Developer</span></div>