Merge lp://qastaging/~renatofilho/qtorganizer5-eds/fix-1423185 into lp://qastaging/qtorganizer5-eds
Proposed by
Renato Araujo Oliveira Filho
| Status: | Merged | ||||
|---|---|---|---|---|---|
| Approved by: | Bill Filler | ||||
| Approved revision: | 74 | ||||
| Merged at revision: | 74 | ||||
| Proposed branch: | lp://qastaging/~renatofilho/qtorganizer5-eds/fix-1423185 | ||||
| Merge into: | lp://qastaging/qtorganizer5-eds | ||||
| Diff against target: |
314 lines (+116/-59) 5 files modified
organizer/qorganizer-eds-engine.cpp (+8/-1) organizer/qorganizer-eds-engine.h (+3/-0) organizer/qorganizer-eds-requestdata.cpp (+12/-39) organizer/qorganizer-eds-requestdata.h (+4/-6) tests/unittest/cancel-operation-test.cpp (+89/-13) |
||||
| To merge this branch: | bzr merge lp://qastaging/~renatofilho/qtorganizer5-eds/fix-1423185 | ||||
| Related bugs: |
|
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Alan Pope 🍺🐧🐱 🦄 (community) | Approve | ||
| PS Jenkins bot | continuous-integration | Needs Fixing | |
|
Review via email:
|
|||
Commit message
Fixed engine crash while destroying and requests still running.
To post a comment you must log in.
Revision history for this message
| PS Jenkins bot (ps-jenkins) wrote | # |
review:
Needs Fixing
(continuous-integration)
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
FAILED: Continuous integration, rev:74
http://
Executed test runs:
FAILURE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://