This issue tracker has been migrated to GitHub ,
and is currently read-only.
For more information,
see the GitHub FAQs in the Python's Developer Guide.
Created on 2008年05月09日 15:27 by mthard, last changed 2022年04月11日 14:56 by admin. This issue is now closed.
| Files | ||||
|---|---|---|---|---|
| File name | Uploaded | Description | Edit | |
| sched.py.patch | mthard, 2008年05月09日 15:27 | Patch to sched.py | ||
| sched.py.patch-r252 | rbp, 2008年05月10日 14:04 | Patch to sched.py on Python 2.5.2 | ||
| Messages (3) | |||
|---|---|---|---|
| msg66480 - (view) | Author: Matt Harden (mthard) | Date: 2008年05月09日 15:27 | |
In the method scheduler.run in the sched module, heapq.heappush is called with single argument (event). It's a two argument function. The correct call would be heapq.heappush(q, event). This problem is probably not frequently experienced because it only occurs if the delay function modifies the queue, deleting the top entry while sched.run() is waiting to execute it. A patch is attached based on the trunk. |
|||
| msg66524 - (view) | Author: Rodrigo Bernardo Pimentel (rbp) (Python committer) | Date: 2008年05月10日 14:04 | |
+1 on the patch. IIRC, there won't be any more bugfix releases for 2.5.x, but, just in case: the patch doesn't work on 2.5 (though the issue lists it as an affected version - and it is!), so I'm uploading a patch for it (svn tag r252). I started writing a test for it, but it's such an obvious and trivial patch that I think it's unnecessary. |
|||
| msg66640 - (view) | Author: Georg Brandl (georg.brandl) * (Python committer) | Date: 2008年05月11日 15:22 | |
Thanks, fixed in r63070, r63071 (2.5). |
|||
| History | |||
|---|---|---|---|
| Date | User | Action | Args |
| 2022年04月11日 14:56:34 | admin | set | github: 47052 |
| 2008年05月11日 15:22:05 | georg.brandl | set | status: open -> closed resolution: fixed messages: + msg66640 nosy: + georg.brandl |
| 2008年05月10日 14:04:17 | rbp | set | files:
+ sched.py.patch-r252 nosy: + rbp messages: + msg66524 |
| 2008年05月09日 15:27:03 | mthard | create | |