From 60d21d58c2dd67a083857aafe9c3d58c4bd0685e Mon Sep 17 00:00:00 2001 From: Lars Uebernickel Date: Thu, 20 Feb 2014 17:08:58 +0100 Subject: Add support for amplified volumes Add a settings key "allow-amplified-volume" which controls whether the volume slider stops at 100% or PA_VOLUME_UI_MAX. unity-control-center will provide a ui for this key. --- src/service.vala | 45 ++++++++++++++++++++++++++++++++++++++------- 1 file changed, 38 insertions(+), 7 deletions(-) (limited to 'src/service.vala') diff --git a/src/service.vala b/src/service.vala index f7d86bb..3a64cb2 100644 --- a/src/service.vala +++ b/src/service.vala @@ -17,7 +17,7 @@ * Lars Uebernickel */ -public class IndicatorSound.Service { +public class IndicatorSound.Service: Object { public Service () { this.settings = new Settings ("com.canonical.indicator.sound"); @@ -54,6 +54,8 @@ public class IndicatorSound.Service { this.notification.set_hint_string ("x-canonical-private-synchronous", "indicator-sound"); } } + + settings.bind ("allow-amplified-volume", this, "allow-amplified-volume", SettingsBindFlags.GET); } public int run () { @@ -71,6 +73,24 @@ public class IndicatorSound.Service { return 0; } + public bool allow_amplified_volume { + get { + return this.max_volume > 1.0; + } + + set { + 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; + } + else { + this.max_volume = 1.0; + } + + this.actions.change_action_state ("volume", this.volume_control.get_volume () / this.max_volume); + } + } + const ActionEntry[] action_entries = { { "root", null, null, "@a{sv} {}", null }, { "scroll", activate_scroll_action, "i", null, null }, @@ -87,6 +107,7 @@ public class IndicatorSound.Service { uint player_action_update_id; Notify.Notification notification; bool syncing_preferred_players = false; + double max_volume = 1.0; const double volume_step_percentage = 0.06; @@ -94,7 +115,7 @@ public class IndicatorSound.Service { int delta = param.get_int32(); /* positive for up, negative for down */ double v = this.volume_control.get_volume () + volume_step_percentage * delta; - this.volume_control.set_volume (v.clamp (0.0, 1.0)); + this.volume_control.set_volume (v.clamp (0.0, this.max_volume)); if (this.notification != null) { string icon; @@ -201,22 +222,32 @@ public class IndicatorSound.Service { void volume_changed (double volume) { var volume_action = this.actions.lookup_action ("volume") as SimpleAction; - volume_action.set_state (new Variant.double (volume)); + volume_action.set_state (new Variant.double (volume / this.max_volume)); this.update_root_icon (); } Action create_volume_action () { - var volume_action = new SimpleAction.stateful ("volume", VariantType.INT32, new Variant.double (this.volume_control.get_volume ())); + /* The action's state is between be in [0.0, 1.0] instead of [0.0, + * max_volume], so that we don't need to update the slider menu item + * every time allow-amplified-volume is changed. Convert between the + * two here, so that we always pass the full range into + * volume_control.set_volume(). + */ + + double volume = this.volume_control.get_volume () / this.max_volume; + + var volume_action = new SimpleAction.stateful ("volume", VariantType.INT32, new Variant.double (volume)); volume_action.change_state.connect ( (action, val) => { - volume_control.set_volume (val.get_double ()); + volume_control.set_volume (val.get_double () * this.max_volume); }); /* activating this action changes the volume by the amount given in the parameter */ volume_action.activate.connect ( (action, param) => { - double v = volume_control.get_volume () + volume_step_percentage * param.get_int32 (); - volume_control.set_volume (v.clamp (0.0, 1.0)); + int delta = param.get_int32 (); + double v = volume_control.get_volume () + volume_step_percentage * delta; + volume_control.set_volume (v.clamp (0.0, this.max_volume)); }); this.volume_control.volume_changed.connect (volume_changed); -- cgit v1.2.3 From 474b7ff95f176c0d1a12df3107c1b21b129b138d Mon Sep 17 00:00:00 2001 From: Lars Uebernickel Date: Thu, 20 Feb 2014 18:22:01 +0100 Subject: Clamp volume when in the action's change_state handler --- src/service.vala | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) (limited to 'src/service.vala') diff --git a/src/service.vala b/src/service.vala index 3a64cb2..87e8b8b 100644 --- a/src/service.vala +++ b/src/service.vala @@ -240,7 +240,8 @@ public class IndicatorSound.Service: Object { var volume_action = new SimpleAction.stateful ("volume", VariantType.INT32, new Variant.double (volume)); volume_action.change_state.connect ( (action, val) => { - volume_control.set_volume (val.get_double () * this.max_volume); + double v = val.get_double () * this.max_volume; + volume_control.set_volume (v.clamp (0.0, this.max_volume)); }); /* activating this action changes the volume by the amount given in the parameter */ -- cgit v1.2.3 From bbecb29d344887879e900319eb96cf19d44bde27 Mon Sep 17 00:00:00 2001 From: Lars Uebernickel Date: Thu, 20 Feb 2014 18:32:14 +0100 Subject: Add comments explaining max_volume a bit better --- src/service.vala | 7 +++++++ 1 file changed, 7 insertions(+) (limited to 'src/service.vala') diff --git a/src/service.vala b/src/service.vala index 87e8b8b..b1b5ba9 100644 --- a/src/service.vala +++ b/src/service.vala @@ -87,6 +87,7 @@ public class IndicatorSound.Service: Object { this.max_volume = 1.0; } + /* Normalize volume, because the volume action's state is [0.0, 1.0], see create_volume_action() */ this.actions.change_action_state ("volume", this.volume_control.get_volume () / this.max_volume); } } @@ -107,6 +108,10 @@ public class IndicatorSound.Service: Object { uint player_action_update_id; Notify.Notification notification; bool syncing_preferred_players = false; + + /* Maximum volume as a scaling factor between the volume action's state and the value in + * this.volume_control. See create_volume_action(). + */ double max_volume = 1.0; const double volume_step_percentage = 0.06; @@ -222,6 +227,8 @@ public class IndicatorSound.Service: Object { void volume_changed (double volume) { var volume_action = this.actions.lookup_action ("volume") as SimpleAction; + + /* Normalize volume, because the volume action's state is [0.0, 1.0], see create_volume_action() */ volume_action.set_state (new Variant.double (volume / this.max_volume)); this.update_root_icon (); -- cgit v1.2.3