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