Message283151
> I believe that I addressed all the comments in the previous review (although its always possible I missed something), and have a new patch with the improvements and fixes.
Thanks Eric! I've left a few more comments in the review. Almost there!
> I also noticed that in asyncio, loop.shutdown_asyncgens() is a coroutinemethod and fixed the markup to reflect that as I previously labeled it a method. The example I added here is as suggested, but it differs from the example in PEP525 in that shutdown_asyncgens() is run in a finally clause, which makes sense to me.
Right, I'll fix the PEP. Thanks! |
|
Date |
User |
Action |
Args |
2016-12-14 01:05:32 | yselivanov | set | recipients:
+ yselivanov, gvanrossum, ncoghlan, ned.deily, Eric Appelt |
2016-12-14 01:05:32 | yselivanov | set | messageid: <1481677532.01.0.818815777257.issue28091@psf.upfronthosting.co.za> |
2016-12-14 01:05:31 | yselivanov | link | issue28091 messages |
2016-12-14 01:05:31 | yselivanov | create | |
|