Merge lp://qastaging/~cjcurran/indicator-sound/core-usage into lp://qastaging/indicator-sound/sound-menu-v2
Proposed by
Conor Curran
| Status: | Merged | ||||||||
|---|---|---|---|---|---|---|---|---|---|
| Merged at revision: | 143 | ||||||||
| Proposed branch: | lp://qastaging/~cjcurran/indicator-sound/core-usage | ||||||||
| Merge into: | lp://qastaging/indicator-sound/sound-menu-v2 | ||||||||
| Diff against target: |
11 lines (+0/-1) 1 file modified
src/pulse-manager.c (+0/-1) |
||||||||
| To merge this branch: | bzr merge lp://qastaging/~cjcurran/indicator-sound/core-usage | ||||||||
| Related bugs: |
|
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Kalle Valo (community) | Approve | ||
|
Review via email:
|
|||
Description of the change
before, I was kicking off a new connection to pulse if the initial attempt failed even though the reconnect flag was passed.
Fixes many duplicate bugs been reported since early lucid.
To post a comment you must log in.
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
I don't know enough about PulseAudio to really understand this. I assume the Pa connection is started somewhere else.
But the patch itself looks safe.