Message377249
Yury,
Why was the PR merged with a new API function `PyGen_Send`?
I explicitly said that any new API function should *not* start with `PyGen`, nor should any function rely on generators and async "coroutines" sharing the same memory layout.
If you disagree with me, please say why, don't just merge the PR.
The name `PyGen` is misleading as it can handle coroutines as well as generators.
There is no performance advantage to only handling these two types.
Worse, it requires that a `PyCoroObject` can always be cast to a `PyGenObject`, preventing better layout of either object in the future.
Would you revert the PR, please. |
|
Date |
User |
Action |
Args |
2020-09-21 10:50:10 | Mark.Shannon | set | recipients:
+ Mark.Shannon, scoder, lukasz.langa, serhiy.storchaka, yselivanov, v2m |
2020-09-21 10:50:10 | Mark.Shannon | set | messageid: <1600685410.16.0.2869436519.issue41756@roundup.psfhosted.org> |
2020-09-21 10:50:10 | Mark.Shannon | link | issue41756 messages |
2020-09-21 10:50:09 | Mark.Shannon | create | |
|