Merge lp://staging/~exarkun/ampoule/simpler-doWork-deferreds into lp://staging/ampoule
Proposed by
Jean-Paul Calderone
| Status: | Merged |
|---|---|
| Merged at revision: | 46 |
| Proposed branch: | lp://staging/~exarkun/ampoule/simpler-doWork-deferreds |
| Merge into: | lp://staging/ampoule |
| Diff against target: |
194 lines (+44/-40) 3 files modified
ampoule/main.py (+1/-1) ampoule/pool.py (+7/-14) ampoule/test/test_process.py (+36/-25) |
| To merge this branch: | bzr merge lp://staging/~exarkun/ampoule/simpler-doWork-deferreds |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| dialtone | Approve | ||
|
Review via email:
|
|||
Description of the change
Started off just trying to simplify _cb_doWork. This involved getting the tests to pass reliably, though. So, the branch implements one minor simplification to _cb_doWork, and changes a handful of tests in order to make them pass reliably on my desktop.
To post a comment you must log in.
Revision history for this message
| dialtone (dialtone) | # |
review:
Approve
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
