Review Request 110390: Do not set Phonon KCM as changed at startup when using PulseAudio

Jan Grulich grulja at gmail.com
Sat May 11 19:46:18 BST 2013


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

Review request for KDE Runtime.


Description
-------

This patch fixes a problem in Phonon KCM which is always set as changed when using PulseAudio. I made two changes. The first change is renaming of ready() signal from AudioSetup class to pulseAudioReady() which is connected to slot in KCM which sets PulseAudio to enabled in Phonon::DevicePreference. The second change is a new signal emitted when GUI is initialized and only when pulseAudioReady() signal was emitted before. This signal is connected to slot in KCM which inserts this widget to KCM and connects changed() signal to KCM changed() slot.


Diffs
-----

  phonon/kcm/audiosetup.h 4887efe 
  phonon/kcm/audiosetup.cpp 35dc4ca 
  phonon/kcm/main.h 277adfe 
  phonon/kcm/main.cpp 5d75cba 

Diff: http://git.reviewboard.kde.org/r/110390/diff/


Testing
-------


Thanks,

Jan Grulich

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20130511/9ee42c79/attachment.htm>


More information about the kde-core-devel mailing list