Merge lp:~cjcurran/indicator-sound/reconnect-bug into lp:indicator-sound/sound-menu-v2
Proposed by
Conor Curran
Status: | Merged |
---|---|
Merged at revision: | 152 |
Proposed branch: | lp:~cjcurran/indicator-sound/reconnect-bug |
Merge into: | lp:indicator-sound/sound-menu-v2 |
Diff against target: |
161 lines (+69/-15) 1 file modified
src/pulse-manager.c (+69/-15) |
To merge this branch: | bzr merge lp:~cjcurran/indicator-sound/reconnect-bug |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Ted Gould (community) | Needs Fixing | ||
Review via email: mp+39987@code.launchpad.net |
Description of the change
fixes the bug attached.
when the daemon dies, the state callback should be notified at which point it should schedule a reconnect request after a 5 second delay.
once the pulse context has been renewed it should try to connect again. If this does not work then it should wait idle until pulse decides to start again.
To post a comment you must log in.
* I think you need to do some sort of back off on the reconnection. It seems like there might be cases where people have disabled Pulse, and it wouldn't be coming back anytime soon.
* I'd only create the hashtable and context in the function, and just call that function at init(). That way there is less likely to be bugs in the future.