Merge lp:~ted/indicator-application/repeating-items into lp:indicator-application/0.4
Proposed by
Ted Gould
Status: | Merged | ||||
---|---|---|---|---|---|
Merged at revision: | 189 | ||||
Proposed branch: | lp:~ted/indicator-application/repeating-items | ||||
Merge into: | lp:indicator-application/0.4 | ||||
Diff against target: |
115 lines (+57/-2) 1 file modified
src/indicator-application.c (+57/-2) |
||||
To merge this branch: | bzr merge lp:~ted/indicator-application/repeating-items | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Conor Curran (community) | Approve | ||
Review via email: mp+53102@code.launchpad.net |
Description of the change
Fixes the duplicate indicators by handling the case when signals are emitted while we have a GetApplications in flight. We basically ignore the signals and recall the GetApplications to ensure that the response is in the correct state. This can delay startup in a few cases, but gaurantees that we won't end up in an inconsistent state.
To post a comment you must log in.