Issue31185
Created on 2017-08-11 16:28 by serhiy.storchaka, last changed 2022-04-11 14:58 by admin. This issue is now closed.
| Pull Requests | |||
|---|---|---|---|
| URL | Status | Linked | Edit |
| PR 3076 | merged | serhiy.storchaka, 2017-08-11 16:33 | |
| PR 3269 | merged | serhiy.storchaka, 2017-09-03 05:47 | |
| Messages (9) | |||
|---|---|---|---|
| msg300171 - (view) | Author: Serhiy Storchaka (serhiy.storchaka) * | Date: 2017-08-11 16:28 | |
Proposed patch fixes miscellaneous errors in asyncio speedup module. It is hard to enumerate they all, but they are mostly NULL dereferencing, reference leaks, and handling exceptions in corner cases. |
|||
| msg300172 - (view) | Author: Yury Selivanov (yselivanov) * | Date: 2017-08-11 16:33 | |
Adding INADA, he is a co-author/reviewer of that code. |
|||
| msg300174 - (view) | Author: Yury Selivanov (yselivanov) * | Date: 2017-08-11 16:36 | |
Huge thanks, Serhiy, btw. |
|||
| msg300175 - (view) | Author: STINNER Victor (vstinner) * | Date: 2017-08-11 16:38 | |
"reference leaks" oh? our Refleaks buildbots didn't catch them. Do your new tests reproduce the bugs, to make sure that we don't leak anymore? |
|||
| msg300176 - (view) | Author: Yury Selivanov (yselivanov) * | Date: 2017-08-11 16:40 | |
> "reference leaks" oh? our Refleaks buildbots didn't catch them. The test suite for Tasks and Futures is pretty extensive, but it obviously doesn't cover some obscure use cases. +1 to make sure that any refleak that was discovered should have a new regression test for it. |
|||
| msg300177 - (view) | Author: Serhiy Storchaka (serhiy.storchaka) * | Date: 2017-08-11 16:51 | |
They are occurred in very obscure cases like calling __init__ multiple times. The patch doesn't contain tests for them. |
|||
| msg301183 - (view) | Author: Serhiy Storchaka (serhiy.storchaka) * | Date: 2017-09-03 05:10 | |
New changeset bca4939d806170c3ca5d05f23710d11a8f1669cf by Serhiy Storchaka in branch 'master': bpo-31185: Fixed miscellaneous errors in asyncio speedup module. (#3076) https://github.com/python/cpython/commit/bca4939d806170c3ca5d05f23710d11a8f1669cf |
|||
| msg301184 - (view) | Author: Serhiy Storchaka (serhiy.storchaka) * | Date: 2017-09-03 06:24 | |
New changeset 98bbeb78e06d5756491705920e72f9721850c727 by Serhiy Storchaka in branch '3.6': bpo-31185: Fixed miscellaneous errors in asyncio speedup module. (#3076) (#3269) https://github.com/python/cpython/commit/98bbeb78e06d5756491705920e72f9721850c727 |
|||
| msg368427 - (view) | Author: Chris Jerdonek (chris.jerdonek) * | Date: 2020-05-08 08:17 | |
FYI, there is a missing Py_DECREF from this change: https://bugs.python.org/issue40559 (though it's an obscure code path as mentioned elsewhere in these comments) |
|||
| History | |||
|---|---|---|---|
| Date | User | Action | Args |
| 2022-04-11 14:58:50 | admin | set | github: 75368 |
| 2020-05-08 08:17:05 | chris.jerdonek | set | nosy:
+ chris.jerdonek messages: + msg368427 |
| 2017-09-03 06:31:28 | serhiy.storchaka | set | status: open -> closed resolution: fixed stage: patch review -> resolved |
| 2017-09-03 06:24:34 | serhiy.storchaka | set | messages: + msg301184 |
| 2017-09-03 05:47:32 | serhiy.storchaka | set | pull_requests: + pull_request3313 |
| 2017-09-03 05:10:16 | serhiy.storchaka | set | messages: + msg301183 |
| 2017-08-11 16:51:08 | serhiy.storchaka | set | messages: + msg300177 |
| 2017-08-11 16:40:31 | yselivanov | set | messages: + msg300176 |
| 2017-08-11 16:38:23 | vstinner | set | messages: + msg300175 |
| 2017-08-11 16:36:30 | yselivanov | set | messages: + msg300174 |
| 2017-08-11 16:33:30 | yselivanov | set | nosy:
+ methane messages: + msg300172 |
| 2017-08-11 16:33:09 | serhiy.storchaka | set | pull_requests: + pull_request3113 |
| 2017-08-11 16:28:18 | serhiy.storchaka | create | |