Merge lp:~tiagosh/history-service/fix-qt52 into lp:history-service
Proposed by
Tiago Salem Herrmann
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Gustavo Pichorim Boiko | ||||
Approved revision: | 130 | ||||
Merged at revision: | 130 | ||||
Proposed branch: | lp:~tiagosh/history-service/fix-qt52 | ||||
Merge into: | lp:history-service | ||||
Diff against target: |
70 lines (+19/-15) 2 files modified
Ubuntu/History/historyeventmodel.cpp (+9/-7) Ubuntu/History/historythreadmodel.cpp (+10/-8) |
||||
To merge this branch: | bzr merge lp:~tiagosh/history-service/fix-qt52 | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Gustavo Pichorim Boiko (community) | Approve | ||
PS Jenkins bot | continuous-integration | Approve | |
Review via email: mp+204728@code.launchpad.net |
Commit message
Check if the model is already empty before removing items in order to avoid an assert() in qt 5.2.
Description of the change
Check if the model is already empty before removing items in order to avoid an assert() in qt 5.2.
To post a comment you must log in.
PASSED: Continuous integration, rev:130 jenkins. qa.ubuntu. com/job/ history- service- ci/99/ jenkins. qa.ubuntu. com/job/ history- service- trusty- amd64-ci/ 26 jenkins. qa.ubuntu. com/job/ history- service- trusty- armhf-ci/ 26 jenkins. qa.ubuntu. com/job/ history- service- trusty- armhf-ci/ 26/artifact/ work/output/ *zip*/output. zip jenkins. qa.ubuntu. com/job/ history- service- trusty- i386-ci/ 26
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild: s-jenkins. ubuntu- ci:8080/ job/history- service- ci/99/rebuild
http://