Review Request 123834: Switch the login sound to Phonon directly...for now

Martin Klapetek martin.klapetek at gmail.com
Tue May 26 09:24:08 UTC 2015


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

(Updated May 26, 2015, 9:24 a.m.)


Status
------

This change has been marked as submitted.


Review request for Plasma.


Changes
-------

Submitted with commit c445f99304bd0190a7a0b83e4700222132bc34d1 by Martin Klapetek to branch master.


Repository: plasma-workspace


Description
-------

With its current architecture, KNotification can cause crashes on logout (and also cause asserts in certain situations, that will be another fix). So in the meantime, this replaces the KNotification-in-a-thread with Phonon directly.

This is exactly what KNotification would do. This is for the time being until the crash on logout is sorted out.

Additionally, this also fixes logout sound which was missing before. This uses normal KNotification as at that point we don't need to be threading or anything, so KNotification is just safe.


Diffs
-----

  CMakeLists.txt 8ffccad 
  ksmserver/CMakeLists.txt c0543e2 
  ksmserver/shutdown.cpp 7600c30 
  ksmserver/startup.cpp f79fd4f 

Diff: https://git.reviewboard.kde.org/r/123834/diff/


Testing
-------

Login sound works as expected, logout sound as well. Also tested by couple other people.


Thanks,

Martin Klapetek

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


More information about the Plasma-devel mailing list