Opened 8 years ago

Closed 8 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:
Author: Launchpad Bug:

Description

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 8 years ago by ralphm

  • Resolution set to fixed
  • Status changed from new to closed

(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 3 years ago by <automation>

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