Review Request 123770: Play the login sound in its own thread

[Martin Klapetek]

> On May 13, 2015, 1:05 p.m., David Edmundson wrote:
> > do you still have any backtraces of the original freeze?
> 
> Martin Klapetek wrote:
>     I never had any, but Hrvoje and Mark had those problems as reported in https://git.reviewboard.kde.org/r/118636/

[14:01:22] <shumski___> mck182: seems your patch is working nicely, at least 2 logins went fine


- Martin


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/123770/#review80290
-----------------------------------------------------------


On May 13, 2015, 12:09 p.m., Martin Klapetek wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/123770/
> -----------------------------------------------------------
> 
> (Updated May 13, 2015, 12:09 p.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Repository: plasma-workspace
> 
> 
> Description
> -------
> 
> This feature was reverted back in the days as somehow it was blocking with PulseAudio startup which delayed the startup sequence quite considerably. This moves the sound playing into its own thread, which should prevent the blocking.
> 
> 
> Diffs
> -----
> 
>   ksmserver/startup.cpp 4155347 
> 
> Diff: https://git.reviewboard.kde.org/r/123770/diff/
> 
> 
> Testing
> -------
> 
> I was never able to reproduce the blocking, so I'm not sure if this actually fixes the problem.
> 
> 
> Thanks,
> 
> Martin Klapetek
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20150513/71155107/attachment.html>