Merge lp:~zsombi/ubuntu-ui-toolkit/alarmmodel-segfault into lp:ubuntu-ui-toolkit/staging
Proposed by
Cris Dywan
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Zsombor Egri | ||||
Approved revision: | 1197 | ||||
Merged at revision: | 1549 | ||||
Proposed branch: | lp:~zsombi/ubuntu-ui-toolkit/alarmmodel-segfault | ||||
Merge into: | lp:ubuntu-ui-toolkit/staging | ||||
Diff against target: |
47 lines (+12/-2) 2 files modified
modules/Ubuntu/Components/plugin/ucalarmmodel.cpp (+9/-2) modules/Ubuntu/Components/plugin/ucalarmmodel.h (+3/-0) |
||||
To merge this branch: | bzr merge lp:~zsombi/ubuntu-ui-toolkit/alarmmodel-segfault | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
PS Jenkins bot | continuous-integration | Approve | |
Zsombor Egri | Approve | ||
Nekhelesh Ramananthan | Pending | ||
Tim Peeters | Pending | ||
Review via email: mp+263940@code.launchpad.net |
This proposal supersedes a proposal from 2015-06-16.
Commit message
Fix AlamrModel segfault when updating elements.
To post a comment you must log in.
Seems good.
Including a regression test would be even better, but may be difficult to write.