Skip to content

[3.7] bpo-33584: Fix several minor bugs in asyncio. (GH-7003) - #7005

Merged
serhiy-storchaka merged 1 commit into
python:3.7from
miss-islington:backport-6655354-3.7
May 20, 2018
Merged

[3.7] bpo-33584: Fix several minor bugs in asyncio. (GH-7003)#7005
serhiy-storchaka merged 1 commit into
python:3.7from
miss-islington:backport-6655354-3.7

Conversation

@miss-islington

@miss-islington miss-islington commented May 20, 2018

Copy link
Copy Markdown
Contributor

Fix the following bugs in the C implementation:

  • get_future_loop() silenced all exceptions raised when look up the get_loop
    attribute, not just an AttributeError.
  • enter_task() silenced all exceptions raised when look up the current task,
    not just a KeyError.
  • repr() was called for a borrowed link in enter_task() and task_step_impl().
  • str() was used instead of repr() in formatting one error message (in
    Python implementation too).
  • There where few reference leaks in error cases.
    (cherry picked from commit 6655354)

Co-authored-by: Serhiy Storchaka storchaka@gmail.com

https://bugs.python.org/issue33584

Fix the following bugs in the C implementation:

* get_future_loop() silenced all exceptions raised when look up the get_loop
  attribute, not just an AttributeError.
* enter_task() silenced all exceptions raised when look up the current task,
  not just a KeyError.
* repr() was called for a borrowed link in enter_task() and task_step_impl().
* str() was used instead of repr() in formatting one error message (in
  Python implementation too).
* There where few reference leaks in error cases.
(cherry picked from commit 6655354)

Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
@miss-islington

Copy link
Copy Markdown
Contributor Author

@serhiy-storchaka: Backport status check is done, and it's a failure ❌ .

1 similar comment
@miss-islington

Copy link
Copy Markdown
Contributor Author

@serhiy-storchaka: Backport status check is done, and it's a failure ❌ .

@miss-islington

Copy link
Copy Markdown
Contributor Author

Thanks, @serhiy-storchaka!

@miss-islington
miss-islington deleted the backport-6655354-3.7 branch May 20, 2018 14:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip news type-bug An unexpected behavior, bug, or error

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants