Message293294
Wouldn't be better to split _GeneratorContextManager on two classes rather than add a boolean argument? Currently _GeneratorContextManager used two different functions -- an one-shot context manager and a decorator that recreates context managers.
Proposed patch makes _GeneratorContextManager an one-shot context manager again (it no longer keep references to arguments), and adds _GeneratorContextManagerDecorator which can be used either as an one-shot context manager (references to arguments are removed after use) or as a decorator (in the last case the gen attribute is not created). |
|
Date |
User |
Action |
Args |
2017-05-09 09:12:24 | serhiy.storchaka | set | recipients:
+ serhiy.storchaka, ncoghlan, Martin.Teichmann |
2017-05-09 09:12:24 | serhiy.storchaka | set | messageid: <1494321144.29.0.525652304123.issue30306@psf.upfronthosting.co.za> |
2017-05-09 09:12:24 | serhiy.storchaka | link | issue30306 messages |
2017-05-09 09:12:23 | serhiy.storchaka | create | |
|