The Sound Menu

Merge lp:~ted/indicator-sound/silent-mode-startup into lp:indicator-sound/rtm-14.09

  1. silent-mode-startup
  2. Merge into trunk.rtm-14.09
Proposed by Ted Gould
Status: Merged
Approved by: Charles Kerr
Approved revision: 466
Merged at revision: 464
Proposed branch: lp:~ted/indicator-sound/silent-mode-startup
Merge into: lp:indicator-sound/rtm-14.09
Diff against target: 21 lines (+3/-1)
1 file modified
src/accounts-service-user.vala (+3/-1)
To merge this branch: bzr merge lp:~ted/indicator-sound/silent-mode-startup
Reviewer Review Type Date Requested Status
Charles Kerr (community) Approve
Review via email: mp+241336@code.launchpad.net

Commit message

Set internal silent mode variable at startup

To post a comment you must log in.
lp:~ted/indicator-sound/silent-mode-startup updated
466. By Ted Gould

Make sure to set the internal variable too

Revision history for this message
Charles Kerr (charlesk) wrote :

LGTM.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/accounts-service-user.vala'
2--- src/accounts-service-user.vala 2014-10-14 17:11:23 +0000
3+++ src/accounts-service-user.vala 2014-12-09 15:56:07 +0000
4@@ -35,6 +35,7 @@
5 return _silentMode;
6 }
7 set {
8+ _silentMode = value;
9 if (syssoundproxy != null)
10 syssoundproxy.silent_mode = value;
11 }
12@@ -211,7 +212,8 @@
13 }
14 });
15
16- this.silentMode = this.syssoundproxy.silent_mode;
17+ this._silentMode = this.syssoundproxy.silent_mode;
18+ this.notify_property("silentMode");
19 } catch (Error e) {
20 this.syssoundproxy = null;
21 warning("Unable to get proxy to system sound settings: %s", e.message);

Subscribers

People subscribed via source and target branches