Like the idea, the more debug info we have the better! https://codereview.appspot.com/131660044/diff/1/asyncio/events.py File asyncio/events.py (right): ...
9 years, 7 months ago
(2014-09-12 22:39:45 UTC)
#1
On 2014/09/12 22:39:45, yselivanov wrote: > Like the idea, the more debug info we have ...
9 years, 7 months ago
(2014-09-17 21:18:45 UTC)
#2
On 2014/09/12 22:39:45, yselivanov wrote:
> Like the idea, the more debug info we have the better!
Cool.
> https://codereview.appspot.com/131660044/diff/1/asyncio/events.py
> File asyncio/events.py (right):
>
> https://codereview.appspot.com/131660044/diff/1/asyncio/events.py#newcode148
> asyncio/events.py:148: pos = 2 if self._cancelled else 1
> Don't like this play with positions. Can we just always append it at the end?
I don't like this code neither, but I don't want to change the representation in
this commit. They are unit tests for repr(TimerHandle). And I like displaying
most important information at the beginning of the representation, to ease
debug.
Issue 131660044: In debug mode, Handle and TimerHandle keeps the callback in their representation after cancel()
Created 9 years, 8 months ago by haypo_gmail
Modified 9 years, 7 months ago
Reviewers: yselivanov
Base URL:
Comments: 1