diff options
| author | Lars Uebernickel <lars.uebernickel@canonical.com> | 2014-10-16 13:31:32 +0000 |
|---|---|---|
| committer | CI bot <ps-jenkins@lists.canonical.com> | 2014-10-16 13:31:32 +0000 |
| commit | 0c656d66748f08e8753243fe591dca22d69bf36c (patch) | |
| tree | aeec93c76d9373772583f9360557b5e0f951dadf | |
| parent | ea570316012b87f04d1de7aff11d2bebff670b97 (diff) | |
| parent | 16a72ae6f4c18c621e16e259bc9122cdaf39eb6b (diff) | |
| download | ayatana-indicator-sound-0c656d66748f08e8753243fe591dca22d69bf36c.tar.gz ayatana-indicator-sound-0c656d66748f08e8753243fe591dca22d69bf36c.tar.bz2 ayatana-indicator-sound-0c656d66748f08e8753243fe591dca22d69bf36c.zip | |
service.vala: don't call set_volume unnecessarily
The allow-amplified-volume setting is bound to a property on the service with the same name. GSettings always sets the target property when the binding is created. The property setter calls set_volume() unconditionally, which lead to a call on every startup of indicator-sound. That wasn't a problem until set_volume() started emitting a notification for volume changes... Fixes: 1381871
Approved by: Ted Gould
| -rw-r--r-- | src/service.vala | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/src/service.vala b/src/service.vala index bd44b0d..0a5725a 100644 --- a/src/service.vala +++ b/src/service.vala @@ -139,6 +139,9 @@ public class IndicatorSound.Service: Object { } set { + if (this.allow_amplified_volume == value) + return; + if (value) { /* from pulse/volume.h: #define PA_VOLUME_UI_MAX (pa_sw_volume_from_dB(+11.0)) */ this.max_volume = (double)PulseAudio.Volume.sw_from_dB(11.0) / PulseAudio.Volume.NORM; |