diff options
author | Serhiy Storchaka <storchaka@gmail.com> | 2012-12-29 19:13:45 (GMT) |
---|---|---|
committer | Serhiy Storchaka <storchaka@gmail.com> | 2012-12-29 19:13:45 (GMT) |
commit | c04957bff3a53ba3d051b7c4148a48ec5238f3cb (patch) | |
tree | 9f65af68a5cc6e41ec70ea0bdd74c8a907fe1e78 /Lib/sched.py | |
parent | e912496c6025581f8f1554c9cda92ae73f2514e5 (diff) | |
download | cpython-c04957bff3a53ba3d051b7c4148a48ec5238f3cb.zip cpython-c04957bff3a53ba3d051b7c4148a48ec5238f3cb.tar.gz cpython-c04957bff3a53ba3d051b7c4148a48ec5238f3cb.tar.bz2 |
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 e523bc1..4b1f7ac 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. |