Message258044
The patch looks good to me, but I'd like to see the error message convey two points:
- the coroutine has already terminated (regardless of how that happened)
- the calling code attempted to resume it anyway
That is, something like "Cannot resume terminated coroutine", rather than specifically referring to "await".
After all, waiting for the result with "await" is only one way to terminate a coroutine - you can also get there with direct calls to next(), send(), throw() and close(). |
|
Date |
User |
Action |
Args |
2016-01-12 00:35:00 | ncoghlan | set | recipients:
+ ncoghlan, gvanrossum, brett.cannon, vstinner, asvetlov, martin.panter, yselivanov |
2016-01-12 00:35:00 | ncoghlan | set | messageid: <1452558900.84.0.195247609489.issue25887@psf.upfronthosting.co.za> |
2016-01-12 00:35:00 | ncoghlan | link | issue25887 messages |
2016-01-12 00:35:00 | ncoghlan | create | |
|