315383 2013-02-18 13:35:45 +0000 mpris volume/play status not in sync with app in kmix applet 2013-12-20 10:24:20 +0000 1 1 2 Applications kmix general 4.3 Arch Linux Linux RESOLVED FIXED https://bugs.kde.org/show_bug.cgi?id=317926 NOR normal --- 0 rockonthemoonfm esken http://commits.kde.org/kmix/0247b50cffa6962e1863316b855c48b290103b45 4.10.3 0 oldest_to_newest 1342941 0 rockonthemoonfm 2013-02-18 13:35:45 +0000 when I play a song in amarok, volume in kmix applet is set to 0 and instead of pause button I have play. anyway, everything works as expected when interacting with it. using phonon-vlc, no pulseaudio installed, front-speakers as output. with pulseaudio installed, this bug is not present. ask for more infos, please. thanks Reproducible: Always 1360366 1 78869 akulichalexander 2013-04-13 18:39:20 +0000 Created attachment 78869 Query volume at end of addMprisControl(). As I tested, this patch fixes issue. 1363593 2 rohan 2013-04-24 11:36:51 +0000 Git commit d136dfc9cac3255d13c191ca95bd8f90e003eb47 by Rohan Garg, on behalf of Alexandr Akulich. Committed on 24/04/2013 at 13:33. Pushed by garg into branch 'master'. KMix: Fix initial volume for MPRIS2. REVIEW: 110044 M +1 -1 backends/mixer_mpris2.cpp http://commits.kde.org/kmix/d136dfc9cac3255d13c191ca95bd8f90e003eb47 1363594 3 rohan 2013-04-24 11:40:26 +0000 Git commit 0247b50cffa6962e1863316b855c48b290103b45 by Rohan Garg, on behalf of Alexandr Akulich. Committed on 24/04/2013 at 13:33. Pushed by garg into branch 'KDE/4.10'. KMix: Fix initial volume for MPRIS2. REVIEW: 110044 M +1 -1 backends/mixer_mpris2.cpp http://commits.kde.org/kmix/0247b50cffa6962e1863316b855c48b290103b45 1419349 4 esken 2013-12-18 21:26:16 +0000 It looks like this bugfix is very very evil. As far as I understand, the unobtrusive line mad->playerIfc->property("Volume") effectively does a synchronous DBUS call. But "Synchronous DBUS calls are evil" (tm). See bug 317926 for a possible massive impact. 1419631 5 esken 2013-12-20 10:24:20 +0000 Git commit 0f7a603e1d327f76dbd6a62d08315b1956794d42 by Christian Esken. Committed on 20/12/2013 at 10:17. Pushed by esken into branch 'master'. Configuration menu is now using Tabs to provide a more standard, convenient and pleasing layout Sound Menu: Play button reflects playback status and shows either "Play" or "Pause" Bugfixes and Features (CCBUGS, as most bugs are already closed). Hint: Some bugs might have actually been alreay fixed ibn KDE4.11 or KDE4.12. Bug ID Severity Summary Related: bug 317926, bug 20, bug 299477, bug 256854, bug 304144, bug 296951, bug 303608, bug 214854, bug 319600 applet volume control position. M +9 -5 CMakeLists.txt M +41 -20 ChangeLog M +5 -1 apps/KMixApp.cpp M +167 -238 apps/kmix.cpp M +8 -13 apps/kmix.h M +3 -0 apps/kmixctrl.cpp M +3 -0 apps/kmixd.cpp M +18 -12 backends/mixer_backend.cpp M +1 -0 backends/mixer_backend.h M +229 -150 backends/mixer_mpris2.cpp M +23 -6 backends/mixer_mpris2.h M +8 -2 backends/mixer_oss.cpp M +1 -1 backends/mixer_pulse.cpp M +9 -9 core/ControlManager.cpp M +102 -15 core/GlobalConfig.cpp M +105 -47 core/GlobalConfig.h M +65 -31 core/kmixdevicemanager.cpp M +9 -4 core/mixdevice.cpp M +4 -12 core/mixdevice.h M +6 -4 core/mixer.cpp M +16 -10 core/mixertoolbox.cpp M +1 -1 core/version.h M +17 -3 core/volume.cpp M +9 -8 core/volume.h M +19 -6 gui/dialogchoosebackends.cpp M +1 -1 gui/dialogchoosebackends.h M +9 -2 gui/dialogviewconfiguration.cpp M +37 -10 gui/guiprofile.cpp M +27 -24 gui/guiprofile.h M +93 -118 gui/kmixdockwidget.cpp M +3 -4 gui/kmixdockwidget.h M +13 -11 gui/kmixerwidget.cpp M +296 -91 gui/kmixprefdlg.cpp M +82 -32 gui/kmixprefdlg.h M +87 -38 gui/mdwslider.cpp M +4 -2 gui/mdwslider.h M +69 -42 gui/viewbase.cpp M +1 -0 gui/viewbase.h M +25 -30 gui/viewdockareapopup.cpp M +1 -1 gui/viewdockareapopup.h M +4 -4 gui/viewsliders.cpp M +1 -1 gui/volumesliderextradata.h http://commits.kde.org/kmix/0f7a603e1d327f76dbd6a62d08315b1956794d42 78869 2013-04-13 18:39:20 +0000 2013-04-13 18:39:20 +0000 Query volume at end of addMprisControl(). mpris2-fix.patch text/plain 604 akulichalexander ZGlmZiAtLWdpdCBhL2JhY2tlbmRzL21peGVyX21wcmlzMi5jcHAgYi9iYWNrZW5kcy9taXhlcl9t cHJpczIuY3BwCmluZGV4IDg4ZTBhODYuLjUwNTIxY2MgMTAwNjQ0Ci0tLSBhL2JhY2tlbmRzL21p eGVyX21wcmlzMi5jcHAKKysrIGIvYmFja2VuZHMvbWl4ZXJfbXByaXMyLmNwcApAQCAtMzc3LDcg KzM3Nyw3IEBAIHZvaWQgTWl4ZXJfTVBSSVMyOjphZGRNcHJpc0NvbnRyb2woUURCdXNDb25uZWN0 aW9uJiBjb25uLCBRU3RyaW5nIGJ1c0Rlc3RpbmF0aW9uCiAJCQljb25uZWN0KG1hZCwgU0lHTkFM KHZvbHVtZUNoYW5nZWQoTVByaXNBcHBkYXRhKixkb3VibGUpKSwgdGhpcywgU0xPVCh2b2x1bWVD aGFuZ2VkKE1QcmlzQXBwZGF0YSosZG91YmxlKSkgKTsKIAogCQkJY29ubi5jb25uZWN0KGJ1c0Rl c3RpbmF0aW9uLCBRU3RyaW5nKCIvUGxheWVyIiksICJvcmcuZnJlZWRlc2t0b3AuTWVkaWFQbGF5 ZXIiLCAiVHJhY2tDaGFuZ2UiLCBtYWQsIFNMT1QodHJhY2tDaGFuZ2VkSW5jb21pbmcoUVZhcmlh bnRNYXApKSApOwotCisJCQl2b2x1bWVDaGFuZ2VkKG1hZCwgbWFkLT5wbGF5ZXJJZmMtPnByb3Bl cnR5KCJWb2x1bWUiKS50b0RvdWJsZSgpKTsKIH0KIAogCg==