Merge lp:~3v1n0/indicator-appmenu/per-window-showmenu+leak-fix into lp:indicator-appmenu/15.10
Proposed by
Marco Trevisan (Treviño)
Status: | Merged |
---|---|
Approved by: | Andrea Azzarone |
Approved revision: | no longer in the source branch. |
Merged at revision: | 273 |
Proposed branch: | lp:~3v1n0/indicator-appmenu/per-window-showmenu+leak-fix |
Merge into: | lp:indicator-appmenu/15.10 |
Diff against target: |
23 lines (+5/-6) 1 file modified
src/indicator-appmenu.c (+5/-6) |
To merge this branch: | bzr merge lp:~3v1n0/indicator-appmenu/per-window-showmenu+leak-fix |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
PS Jenkins bot (community) | continuous-integration | Approve | |
Andrea Azzarone (community) | Approve | ||
Review via email: mp+273105@code.launchpad.net |
Commit message
IndicatorAppmenu: only emit snow-now-changed signal for menus belonging to the window that requested
Also fix a memory leak we had (entries list had to be free'd)
To post a comment you must log in.
PASSED: Continuous integration, rev:276 jenkins. qa.ubuntu. com/job/ indicator- appmenu- ci/28/ jenkins. qa.ubuntu. com/job/ indicator- appmenu- wily-amd64- ci/3 jenkins. qa.ubuntu. com/job/ indicator- appmenu- wily-armhf- ci/3 jenkins. qa.ubuntu. com/job/ indicator- appmenu- wily-armhf- ci/3/artifact/ work/output/ *zip*/output. zip
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
deb: http://
Click here to trigger a rebuild: s-jenkins. ubuntu- ci:8080/ job/indicator- appmenu- ci/28/rebuild
http://