Opened 14 years ago

Closed 14 years ago

#2222 defect closed fixed (fixed)

t.i.task.Clock.advance should set the call.called attribute.

Reported by: ralphm Owned by:
Priority: normal Milestone:
Component: core Keywords:
Cc: ralphm Branch:


If a DelayedCall produced by twisted.internet.task.Clock.callLater has its cancel() method called after its callback function has been called, the following exception is raised:

exceptions.ValueError: list.remove(x): x not in list

This is quite a cryptic error.

Setting call.called to 1 before calling the callback makes the cancel() call raise the proper AlreadyCalled exception.

Change History (2)

comment:1 Changed 14 years ago by ralphm

Resolution: fixed
Status: newclosed

(In [18644]) Add timeout behaviour to xmpp iq requests and make t.i.Clock set call.called.

Author: ralphm Reviewer: exarkun Fixes #2218 and #2222

comment:2 Changed 10 years ago by <automation>

Owner: ralphm deleted
Note: See TracTickets for help on using tickets.