diff options
author | Serhiy Storchaka <storchaka@gmail.com> | 2012-12-29 21:13:45 +0200 |
---|---|---|
committer | Serhiy Storchaka <storchaka@gmail.com> | 2012-12-29 21:13:45 +0200 |
commit | 5f53162c2f62cefa1348724377d8217e7a25c195 (patch) | |
tree | 07389fe73c3d2ad4935bcbf56e964ed312066742 /Lib/sched.py | |
parent | c23ab666527c28edf0057d7b131075515eaf16a2 (diff) | |
download | cpython-5f53162c2f62cefa1348724377d8217e7a25c195.tar.gz |
Issue #16641: Fix default values of sched.scheduler.enter arguments were modifiable.
Diffstat (limited to 'Lib/sched.py')
-rw-r--r-- | Lib/sched.py | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/Lib/sched.py b/Lib/sched.py index e523bc1c0d..4b1f7ac784 100644 --- a/Lib/sched.py +++ b/Lib/sched.py @@ -50,6 +50,8 @@ class Event(namedtuple('Event', 'time, priority, action, argument, kwargs')): def __gt__(s, o): return (s.time, s.priority) > (o.time, o.priority) def __ge__(s, o): return (s.time, s.priority) >= (o.time, o.priority) +_sentinel = object() + class scheduler: def __init__(self, timefunc=_time, delayfunc=time.sleep): @@ -60,19 +62,21 @@ class scheduler: self.timefunc = timefunc self.delayfunc = delayfunc - def enterabs(self, time, priority, action, argument=[], kwargs={}): + def enterabs(self, time, priority, action, argument=(), kwargs=_sentinel): """Enter a new event in the queue at an absolute time. Returns an ID for the event which can be used to remove it, if necessary. """ + if kwargs is _sentinel: + kwargs = {} with self._lock: event = Event(time, priority, action, argument, kwargs) heapq.heappush(self._queue, event) return event # The ID - def enter(self, delay, priority, action, argument=[], kwargs={}): + def enter(self, delay, priority, action, argument=(), kwargs=_sentinel): """A variant that specifies the time as a relative time. This is actually the more commonly used interface. |